Skip to content

Remove ObStorageLoggerManager and move slog lifecycle into storage meta service#1051

Open
hnwyllmm wants to merge 3 commits into
masterfrom
task/2026070900117293763
Open

Remove ObStorageLoggerManager and move slog lifecycle into storage meta service#1051
hnwyllmm wants to merge 3 commits into
masterfrom
task/2026070900117293763

Conversation

@hnwyllmm

Copy link
Copy Markdown
Member

Task Description

Remove ObStorageLoggerManager and redistribute its responsibilities to the components that own them.

Solution Description

  • Move server slogger ownership and lifecycle management to ObServerStorageMetaService.
  • Pass slog initialization settings directly to ObStorageLogger::init.
  • Move disk reservation logic and SLOG_RESERVED_DISK_SIZE to ObServerStorageMetaService.
  • Move slog item and buffer allocation/freeing into ObStorageLogger.
  • Replace Manager-based tenant slog path access with Router-root-based path construction.
  • Update checkpoint, meta-store, block-manager, and unit tests for the new ownership model.

Passed Regressions

Upgrade Compatibility

Other Information

  • Related links:
    • [DIMA-2026070900117293763](

Release Note

@hnwyllmm

Copy link
Copy Markdown
Member Author

The Dima issue is about changing the standard log buffer to be allocated on-demand.

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.

1 participant