Skip to content

Conversation

@discord9
Copy link
Contributor

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

What's changed and what's your intention?

as title, add value_from_stats for <count/max/min>_state so can eval from statistics

also change has_predicate to consider partition exprs, although not sure if that's correct

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

@discord9 discord9 requested review from a team, evenyag, v0y4g3r and waynexia as code owners October 20, 2025 07:04
@github-actions github-actions bot added size/M docs-not-required This change does not impact docs. labels Oct 20, 2025
@discord9 discord9 changed the title fix: count_state use stat to eval fix: count_state use stat to eval&predicate w/out region Oct 21, 2025
@discord9 discord9 force-pushed the fix/part_count_start branch from d27dd2f to 6bb8f1b Compare October 21, 2025 07:23
@discord9 discord9 force-pushed the fix/part_count_start branch from 6bb8f1b to 5d05445 Compare October 21, 2025 08:06
@github-actions github-actions bot added size/L and removed size/M labels Oct 21, 2025
@MichaelScofield MichaelScofield added this pull request to the merge queue Oct 27, 2025
Merged via the queue into GreptimeTeam:main with commit 68247fc Oct 27, 2025
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required This change does not impact docs. size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants