Skip to content

Conversation

@dqhl76
Copy link
Collaborator

@dqhl76 dqhl76 commented Nov 17, 2025

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

Summary

chore: add more logs to cover aggregate spill

Write aggregate spill finished(exchange): (location: _query_spill/test_tenant/019a9116f3a878e0abb4fee6ef47bdd0_2/jjTyXB207Mwanzmopdjxe3, bytes: 4740, rows: 3, buckets_count: 3, elapsed: 65.501583ms)

Read aggregate spill finished: (bucket: 97, location: _query_spill/test_tenant/019a911d28797a429c58d472e5060af3_2/N2xeVM9mmBEpqTXCE2OIb4, bytes: 86, rows: 1, elapsed: 179.958µs)


Read aggregate 100/160 spilled task, elapsed: 8.062166ms

Read aggregate 100/160 spilled task, elapsed: 6.031625ms

Read aggregate finished: (bucket: 127, total read count: 160, total bytes: 597568, total elapsed: 792ms)

Read aggregate finished: (bucket: 126, total read count: 160, total bytes: 598592, total elapsed: 775ms)

chore: eliminate warnings from sqllogictests for tpch

warn: column type not match, actual: [Any, Any, Any], expected: [Integer, Integer], loc: tests/sqllogictests/suites

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 17, 2025
@dqhl76 dqhl76 marked this pull request as ready for review November 18, 2025 02:32
@dqhl76 dqhl76 requested a review from zhang2014 November 18, 2025 02:41
@dqhl76 dqhl76 merged commit d24a071 into databendlabs:main Nov 18, 2025
87 checks passed
@dqhl76 dqhl76 deleted the add-logs branch November 18, 2025 05:27
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