Skip to content

Conversation

@forsaken628
Copy link
Collaborator

@forsaken628 forsaken628 commented Nov 5, 2025

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

Summary

parts of #17964

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

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):

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 Nov 5, 2025
@forsaken628
Copy link
Collaborator Author

before:

019a51b59d1374d38f1c24360bb5a216 2025-11-05T09:51:15.071706+08:00  INFO databend_query::pipelines::processors::transforms::aggregator::transform_single_key: transform_single_key.rs:172 [TRANSFORM-AGGREGATOR] Single key aggregation completed: 25655213 → 1 rows in 27.57s (real: 27.05s),...

after:

019a51b59d1374d38f1c24360bb5a216 2025-11-05T09:51:15.071706+08:00  INFO [TRANSFORMS]: transform_single_key.rs:172 [TRANSFORM-AGGREGATOR] Single key aggregation completed: 25655213 → 1 rows in 27.57s (real: 27.05s),...

NEED HELP
I'm not sure of the specifics of the mapping table yet, but I'd recommend doing the vast majority of the mapping table in one go, rather than making piecemeal changes over and over again

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.

1 participant