Skip to content

Conversation

@morningman
Copy link
Contributor

@morningman morningman commented Jan 27, 2021

Proposed changes

Previously, we introduced an optimization logic for the aggr table,
that is, in the case of only one rowset and nonoverlapping,
the data can be read directly without merging.
But this logic has bugs.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

Checklist

Previously, we introduced an optimization logic for the aggr table,
that is, in the case of only one rowset and nonoverlapping,
the data can be read directly without merging.
But this logic has bugs.
@morningman morningman added kind/fix Categorizes issue or PR as related to a bug. area/storage Issues or PRs related to storage engine labels Jan 27, 2021
@morningman morningman self-assigned this Jan 27, 2021
Copy link
Member

@yangzhg yangzhg left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman morningman added the approved Indicates a PR has been approved by one committer. label Feb 3, 2021
@morningman morningman merged commit ea7f61e into apache:master Feb 4, 2021
Hastyshell pushed a commit to Hastyshell/doris that referenced this pull request Nov 12, 2025
…ile.releaseMemory() apache#57257 (apache#5499)

cherry-picks from apache#57257
Related to apache#5307

Co-authored-by: minghong <zhouminghong@selectdb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/storage Issues or PRs related to storage engine kind/fix Categorizes issue or PR as related to a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duplicate rows in Aggregate Key Table

2 participants