CalvinKirs opened a new pull request, #64732: URL: https://github.com/apache/doris/pull/64732
## Proposed changes The `apache_hdfs_broker` daemon under `fs_brokers/` has not been modified for nearly two years, and users have migrated to the built-in S3 / HDFS TVF, so the standalone broker component is no longer needed. The in-tree SDKs under `sdk/` (`go-doris-sdk`, `java-doris-sdk`) are now maintained out-of-tree in a separate `doris-sdk` repo and no longer need to live here. ### Removed - `fs_brokers/apache_hdfs_broker/` — broker daemon module - `sdk/` — `go-doris-sdk` + `java-doris-sdk` - `docker/runtime/broker/` — broker daemon docker image, and the `hdfs-broker-doris` compose demo ### Build / CI / docker references cleaned up `build.sh` (`--broker` / `BUILD_BROKER`), `build-for-release.sh`, `.github/workflows/build-extension.yml` (`build-broker` job), `.github/workflows/code-checks.yml`, `.gitignore`, `docker/runtime/docker-build.sh`, `docker/runtime/all-in-one/Dockerfile` + `entry_point.sh`, `docker/runtime/README.md`, and `threat-model.md`. ### Kept (intentionally out of scope) - `fs_brokers/cdc_client/` (not a broker; still built into BE) - All FE/BE broker **functionality**: `BrokerMgr`, `BrokerDesc`, broker load, `fe-filesystem-broker`, the BE filesystem broker module, and `PaloBrokerService.thrift` This PR only removes the standalone broker daemon component and the in-tree SDK sources; the broker load/storage feature in FE/BE is unchanged, so there is no metadata/edit-log compatibility impact. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
