Uh oh!
There was an error while loading. Please reload this page.
[fix](docker) fix be conf newline - #60060
Conversation
hello-stephen
commented
Jan 20, 2026
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
There was a problem hiding this comment.
Pull request overview
This PR improves the clarity of automatically added configuration entries in Docker entrypoint scripts for Doris FE and BE components by replacing blank separator lines with descriptive header comments.
Changes:
- Replaced empty line separators with descriptive header comments ("########## doris-operator automatically adds ##########") before automatically added configuration entries
- Added a missing header comment in the BE entrypoint when be.conf is copied from configmap
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docker/runtime/fe/resource/fe_disaggregated_entrypoint.sh | Replaced blank line with descriptive header comment before cluster configuration additions |
| docker/runtime/be/resource/be_disaggregated_entrypoint.sh | Replaced blank line with descriptive header comment and added header before deploy_mode when be.conf is copied from configmap |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hello-stephen
commented
Jan 20, 2026
skip buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Uh oh!
There was an error while loading. Please reload this page.
…7561) Cherry-pick all docker/runtime/ changes from apache/doris master that are missing in branch-hotfix-selectdb-doris-3.1-hubspot. Includes changes from PRs: apache#42068apache#42208apache#42564apache#42605apache#43040apache#43085apache#43200apache#45503apache#45926apache#46387apache#47342apache#47837apache#48183apache#48839apache#48898apache#49343apache#49874apache#50247apache#50536apache#50604apache#50745apache#50922apache#51357apache#51693apache#54321apache#54392apache#54950apache#55299apache#55548apache#55615apache#55671apache#55824apache#56471apache#56757apache#56917apache#57113apache#57236apache#57345apache#57374apache#57695apache#58229apache#58565apache#58638apache#59039apache#59292apache#59480apache#59553apache#60060apache#60458 ## Proposed changes Issue Number: close #xxx <!--Describe your changes.--> Co-authored-by: catpineapple <catpineapple@users.noreply.github.com>
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)