Skip to content

Don't store whole manifest file entries for position deletes in object info#85740

Merged
divanik merged 2 commits intomasterfrom
divanik/small_inprovement_in_position_deletes
Aug 20, 2025
Merged

Don't store whole manifest file entries for position deletes in object info#85740
divanik merged 2 commits intomasterfrom
divanik/small_inprovement_in_position_deletes

Conversation

@divanik
Copy link
Copy Markdown
Contributor

@divanik divanik commented Aug 16, 2025 •
edited
Loading

Changelog category

  • Not for changelog (changelog entry is not required)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Aug 16, 2025 •
edited
Loading

Workflow [PR], commit [2eb12ee]

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Aug 16, 2025
@kssenii kssenii self-assigned this Aug 18, 2025
@divanik divanik changed the title Don't store whole manifest file entries in object info Don't store whole manifest file entries for position deletes in object info Aug 18, 2025
@divanik
Copy link
Copy Markdown
Contributor Author

divanik commented Aug 20, 2025

https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=85740&sha=latest&name_0=PR&name_1=Integration+tests+%28amd_asan%2C+old+analyzer%2C+3%2F6%29 is flaky

@divanik divanik added this pull request to the merge queue Aug 20, 2025
Merged via the queue into master with commit 818134f Aug 20, 2025
119 of 122 checks passed
@divanik divanik deleted the divanik/small_inprovement_in_position_deletes branch August 20, 2025 16:04
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kssenii kssenii kssenii approved these changes

Assignees

@kssenii kssenii

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@divanik @kssenii @robot-ch-test-poll1