Skip to content

Fix extra overflow row#12747

Merged
KochetovNicolai merged 2 commits into20.4from
fix-extra-overflow-row
Jul 24, 2020
Merged

Fix extra overflow row#12747
KochetovNicolai merged 2 commits into20.4from
fix-extra-overflow-row

Conversation

@KochetovNicolai
Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix possible extra overflow row in data which could appear for queries WITH TOTALS.

@KochetovNicolai KochetovNicolai added pr-bugfix Pull request with bugfix, not backported by default pr-no-backport labels Jul 24, 2020
/// so we can't enable processors for InterpreterSelectQuery here.
auto stream = interpreter.execute().in;
auto source = std::make_shared<SourceFromInputStream>(std::move(stream));
auto source = std::make_shared<SourceFromInputStream>(std::move(stream), processed_stage == QueryProcessingStage::WithMergeableState);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing comment about force_add_aggregating_info in SourceFromInputStream.

@KochetovNicolai KochetovNicolai merged commit da12e29 into 20.4 Jul 24, 2020
@KochetovNicolai KochetovNicolai deleted the fix-extra-overflow-row branch July 24, 2020 12:30
KochetovNicolai added a commit that referenced this pull request Jul 24, 2020
Fix extra overflow row

(cherry picked from commit da12e29)
@alexey-milovidov alexey-milovidov self-assigned this Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default pr-no-backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants