Split StatusSchema and RangeLock from ManagementAPI#13380
Split StatusSchema and RangeLock from ManagementAPI#13380tclinkenbeard-oai wants to merge 1 commit into
StatusSchema and RangeLock from ManagementAPI#13380Conversation
tclinkenbeard-oai
left a comment
There was a problem hiding this comment.
Generated by Codex.
What is it trying to do?
Split the RangeLock and StatusSchema implementations and declarations out of ManagementAPI.cpp/.h into focused files, reducing ManagementAPI.cpp compile cost and replacing transitive includes with explicit owning headers.
Is it correct?
Yes, based on code inspection. The moved RangeLock and StatusSchema function bodies match the originals aside from making previously file-local helpers anonymous-namespace symbols. The declarations keep the same signatures, all in-tree callers include the new headers, and fdb_find_sources picks up both new .cpp files.
No serialized fields, durable formats, protocol boundaries, status output, hot-path algorithms, async behavior, assertions, trace events, or simulation hooks changed.
I did not run builds or tests. Public CI currently has clang-format passing; the Windows Boost check, FoundationDB builders, and Cluster Tests are still pending, with no reported failures.
Are there bugs?
I did not find any correctness bugs.
Are there omissions?
None that I think block this. No new behavioral tests are needed for a body-preserving extraction; the remaining risk is platform-specific compile/link integration, which pending CI should cover.
Are there better ways of doing things?
No material alternative. Moving the declarations to the owning headers and updating consumers to include those headers directly is the appropriate split.
Should this CL be LGTMd?
Yes, LGTM from code inspection. I would still wait for the pending CI checks before merging.
Result of foundationdb-pr-clang-ide on Linux RHEL 9
|
Result of foundationdb-pr-macos-m1 on macOS 14.x
|
Result of foundationdb-pr-clang-arm on Linux RHEL 9
|
Result of foundationdb-pr-clang on Linux RHEL 9
|
Result of foundationdb-pr on Linux RHEL 9
|
Result of foundationdb-pr-cluster-tests on Linux RHEL 9
|
Result of foundationdb-pr-macos on macOS 14.x
|
Result of foundationdb-pr-clang on Linux RHEL 9
|
Result of foundationdb-pr on Linux RHEL 9
|
This PR is a straightforward refactoring to shrink the size of
ManagementAPI.cpp, which often takes a long time to compile.StatusSchemaandRangeLockimplementations are moved to separate files.