Skip to content

feat(inkless:metadata): Remove topic already diskless from InitDisklessLog Controller API#547

Merged
jeqo merged 1 commit intomainfrom
giuseppelillo/diskless-migration-safeguards
Mar 20, 2026
Merged

feat(inkless:metadata): Remove topic already diskless from InitDisklessLog Controller API#547
jeqo merged 1 commit intomainfrom
giuseppelillo/diskless-migration-safeguards

Conversation

@giuseppelillo
Copy link
Copy Markdown
Contributor

The previous check was not correct, as it discarded topics that have diskless.enable=true but are not migrated yet.
The same type of check is already done for each partition, but checking instead that the partition does not have disklessStartOffset already set.

…ssLog Controller API

The previous check was not correct, as it discarded topics that have
diskless.enable=true but are not migrated yet.
The same type of check is already done for each partition, but checking instead
that the partition does not have disklessStartOffset already set.
@giuseppelillo giuseppelillo force-pushed the giuseppelillo/diskless-migration-safeguards branch from 86d9902 to 34eedd6 Compare March 20, 2026 16:39
@jeqo jeqo merged commit 501b2c0 into main Mar 20, 2026
4 checks passed
@jeqo jeqo deleted the giuseppelillo/diskless-migration-safeguards branch March 20, 2026 17:49
AnatolyPopov pushed a commit that referenced this pull request Mar 23, 2026
…ssLog Controller API (#547)

The previous check was not correct, as it discarded topics that have
diskless.enable=true but are not migrated yet.
The same type of check is already done for each partition, but checking instead
that the partition does not have disklessStartOffset already set.

(cherry picked from commit 501b2c0)
jeqo pushed a commit that referenced this pull request Mar 26, 2026
…ssLog Controller API (#547)

The previous check was not correct, as it discarded topics that have
diskless.enable=true but are not migrated yet.
The same type of check is already done for each partition, but checking instead
that the partition does not have disklessStartOffset already set.
jeqo pushed a commit that referenced this pull request Mar 26, 2026
…ssLog Controller API (#547)

The previous check was not correct, as it discarded topics that have
diskless.enable=true but are not migrated yet.
The same type of check is already done for each partition, but checking instead
that the partition does not have disklessStartOffset already set.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants