feat(promql): parameterize lookback

[etolbakov]

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

#3453

What's changed and what's your intention?

Parameterize lookback and allow it to propagate from TQL

Checklist

• I have written the necessary rustdoc comments.
• I have added the necessary unit tests and integration tests.
• This PR does not require documentation updates.

[tisonkun]

Seems we need some integration tests.

Does this patch still need some changes on the parser side?

[etolbakov]

I think it's more or less ready.
Good point about an intergration test (I was planning to add sqlness as well).
Hopefully will push it in the end of this week

[codecov]

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 85.04%. Comparing base (8779524) to head (934bbcf).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3630      +/-   ##
==========================================
- Coverage   85.38%   85.04%   -0.34%     
==========================================
  Files         963      963              
  Lines      160719   160730      +11     
==========================================
- Hits       137231   136700     -531     
- Misses      23488    24030     +542     

[etolbakov]

@waynexia @tisonkun
Could you please take a look when you have time?

[tisonkun]

Generally looks good. Suggest we convey to the HTTP endpoints so we don't leave more TODOs.

[tisonkun]

LGTM.

Further update gRPC codepath and lift step param in HTTP param can be follow-ups.

[etolbakov]

Generally looks good. Suggest we convey to the HTTP endpoints so we don't leave more TODOs.

@tisonkun Thank you for the general feedback! I reflected on that and I'd like to spend a bit more time and deliver something useful rather then a half-baked solution.

As for grpc change I'll try making adjustment in greptime-proto repo

[tisonkun]

@etolbakov Cool! You can ping me as a reviewer when you submit a patch there.

[tisonkun]

error: the lock file /home/runner/work/greptimedb/greptimedb/Cargo.lock needs to be updated but --locked was passed to prevent this
If you want to try to generate the lock file without accessing the network, remove the --locked flag and use --offline instead.

@etolbakov you may update the lock file locally. This should be automatically updated if you're using VSCode or RustRover. Or you can run cargo build without --locked.

[tisonkun]

Will this patch close #3453?

[waynexia]

This is very neat, thanks @etolbakov 🚀

Let's merge this after CI passes.