Skip to content

Conversation

@dqhl76
Copy link
Collaborator

@dqhl76 dqhl76 commented Dec 25, 2025

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

part of #19155 (part 1)

Add shared aggregation statistics for new partial/final aggregators and expose hash index resize count.

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Not run (not requested)

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe): Observability

This change is Reviewable

@github-actions github-actions bot added the pr-chore this PR only has small changes that no need to record, like coding styles. label Dec 25, 2025
@dqhl76
Copy link
Collaborator Author

dqhl76 commented Dec 25, 2025

INFO [TRANSFORMS]: statistics.rs:56 [TRANSFORM-AGGREGATOR][NewFinalAggregate] Aggregation completed: 195025 → 195025 rows in 0.64s (real: 0.12s), throughput: 306.63 thousand rows/sec, 4.68 MiB/sec, total: 2.98 MiB, hash index resizes: 3

@dqhl76 dqhl76 changed the title chore(query): add stats for new aggregators chore(query): add stats for new final aggregators Dec 25, 2025
@dqhl76 dqhl76 marked this pull request as ready for review December 25, 2025 07:56
@dqhl76 dqhl76 requested a review from zhang2014 December 25, 2025 07:57
@zhang2014 zhang2014 merged commit 51b7839 into databendlabs:main Dec 25, 2025
98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-chore this PR only has small changes that no need to record, like coding styles.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants