Yukang-Lian opened a new pull request, #64110:
URL: https://github.com/apache/doris/pull/64110
### What problem does this PR solve?
Issue Number: None
Related PR: #64016
Problem Summary: PR #64016 moved backend server startup behind OSS starter
implementations. The OSS backend thrift starter stopped and released the
ThriftServer but kept its BaseBackendService reference alive after join. That
service owns AgentServer and ReportWorker threads, so callers that rely on the
starter lifecycle can destroy ExecEnv and StorageEngine before ReportWorker
exits. Release the backend service in join after releasing the thrift server.
### Release note
None
### Check List (For Author)
- Test: Unit Test
-
PATH=/mnt/disk2/lianyukang/ldb_toolchain_v16/bin:/data/data14/lianyukang/ldb_toolchain_v16/bin:$PATH
CLANG_FORMAT_BINARY=/data/data14/lianyukang/ldb_toolchain_v16/bin/clang-format-16
build-support/clang-format.sh
-
PATH=/mnt/disk2/lianyukang/ldb_toolchain_v16/bin:/data/data14/lianyukang/ldb_toolchain_v16/bin:$PATH
CLANG_FORMAT_BINARY=/data/data14/lianyukang/ldb_toolchain_v16/bin/clang-format-16
build-support/check-format.sh
- git diff --check HEAD~1..HEAD
-
PATH=/mnt/disk2/lianyukang/ldb_toolchain_v16/bin:/data/data14/lianyukang/ldb_toolchain_v16/bin:$PATH
./run-be-ut.sh --run --filter=BeServerStarterFactoryTest.* -j80
- Behavior changed: No
- Does this need documentation: No
--
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]
