Skip to content

Conversation

@hust-hhb
Copy link
Contributor

@hust-hhb hust-hhb commented Nov 5, 2024

When geting delete bitmap fail, it will retry, howerve the GetDeleteBitmapResponse doesn't clear now, which will lead to the delete bitmap data of last request residual in GetDeleteBitmapResponse and return to be, it a wrong delete bitmap data and may cause be core.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@hust-hhb
Copy link
Contributor Author

hust-hhb commented Nov 5, 2024

run buildall

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2024

clang-tidy review says "All clean, LGTM! 👍"

Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Nov 6, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2024

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2024

PR approved by anyone and no changes requested.

Copy link
Contributor

@zhannngchen zhannngchen left a comment

Choose a reason for hiding this comment

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

LGTM

@dataroaring dataroaring merged commit 6c41a3e into apache:master Nov 6, 2024
github-actions bot pushed a commit that referenced this pull request Nov 6, 2024
…ete bitmap fail and retry (#43261)

When geting delete bitmap fail, it will retry, howerve the
GetDeleteBitmapResponse doesn't clear now, which will lead to the delete
bitmap data of last request residual in GetDeleteBitmapResponse and
return to be, it a wrong delete bitmap data and may cause be core.
dataroaring pushed a commit that referenced this pull request Nov 6, 2024
…g delete bitmap fail and retry (#43358)

Problem Summary:
pr #43261 doesn't clear GetDeleteBitmapResponse correctly, this pr fix
this problem
dataroaring pushed a commit that referenced this pull request Nov 7, 2024
…n geting delete bitmap fail and retry (#43297)

Cherry-picked from #43261

Co-authored-by: huanghaibin <284824253@qq.com>
dataroaring pushed a commit that referenced this pull request Nov 8, 2024
…g delete bitmap fail and retry (#43457)

pr #43261 doesn't clear
GetDeleteBitmapResponse correctly, this pr fix this problem
pick pr #43358

Co-authored-by: huanghaibin <huanghaibin@selectdb.com>
dataroaring pushed a commit that referenced this pull request Dec 5, 2024
…` when geting delete bitmap update lock fail and retry (#44975)

Similar to #43261,
`GetDeleteBitmapUpdateLockResponse` should be cleared after
`get_delete_bitmap_update_lock` fails on MS. Otherwise BE may get staled
compaction cnts and wrongly skip to sync rowsets before
`update_delete_bitmap()` thus causing duplicate keys problem.
github-actions bot pushed a commit that referenced this pull request Dec 5, 2024
…` when geting delete bitmap update lock fail and retry (#44975)

Similar to #43261,
`GetDeleteBitmapUpdateLockResponse` should be cleared after
`get_delete_bitmap_update_lock` fails on MS. Otherwise BE may get staled
compaction cnts and wrongly skip to sync rowsets before
`update_delete_bitmap()` thus causing duplicate keys problem.
BiteTheDDDDt pushed a commit to BiteTheDDDDt/incubator-doris that referenced this pull request Feb 7, 2025
…n geting delete bitmap fail and retry (apache#43297)

Cherry-picked from apache#43261

Co-authored-by: huanghaibin <284824253@qq.com>
BiteTheDDDDt pushed a commit to BiteTheDDDDt/incubator-doris that referenced this pull request Feb 7, 2025
…g delete bitmap fail and retry (apache#43358)

Problem Summary:
pr apache#43261 doesn't clear GetDeleteBitmapResponse correctly, this pr fix
this problem
hubgeter pushed a commit to hubgeter/doris that referenced this pull request Mar 12, 2025
…` when geting delete bitmap update lock fail and retry (apache#44975)

Similar to apache#43261,
`GetDeleteBitmapUpdateLockResponse` should be cleared after
`get_delete_bitmap_update_lock` fails on MS. Otherwise BE may get staled
compaction cnts and wrongly skip to sync rowsets before
`update_delete_bitmap()` thus causing duplicate keys problem.
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. dev/3.0.3-merged p0_b reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants