yagagagaga opened a new pull request, #53147:
URL: https://github.com/apache/doris/pull/53147
### What problem does this PR solve?
To prevent users from accidentally modifying the BE storage directory, I've
added cluster_id verification logic in this PR.
If the cluster IDs under each storage directory are different, the following
error will be thrown:
```
W20250712 14:00:35.846499 347667 cloud_storage_engine.cpp:233] fail to check
cluster id, error: [INTERNAL_ERROR]All root paths must have the same cluster
id, but you have different cluster ids: 12345678, 12345680
E20250712 14:00:35.846525 347667 exec_env_init.cpp:359] Fail to open
StorageEngine, res=[INTERNAL_ERROR]All root paths must have the same cluster
id, but you have different cluster ids: 12345678, 12345680
```
If be.conf is configured with a cluster ID that differs from the one stored
under the storage directory, the following error will occur:
```
W20250712 14:02:25.414953 389878 cloud_storage_engine.cpp:1193] cluster id
not equal, error: [CORRUPTION]multiple cluster ids is not equal.
config::cluster_id=123, storage path cluster_id=12345678
E20250712 14:02:25.415001 389878 exec_env_init.cpp:359] Fail to open
StorageEngine, res=[CORRUPTION]multiple cluster ids is not equal.
config::cluster_id=123, storage path cluster_id=12345678
```
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [x] No need to test or manual test. Explain why:
- [x] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [x] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
