[Enhance](fe) Support setting initial root password when FE firstly launch #27438
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WinkerDu
force-pushed
the
master-initial-root-pswd
branch
from
0509124 to
da17119
Compare
Contributor
Author
|
run buildall |
WinkerDu
force-pushed
the
master-initial-root-pswd
branch
from
da17119 to
2216cee
Compare
Contributor
Author
|
run buildall |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
Contributor
Author
|
cc @morningman @chenlinzhong @luozenglin Please have a review, thx |
WinkerDu
force-pushed
the
master-initial-root-pswd
branch
from
2216cee to
f5fc04d
Compare
Contributor
Author
|
run buildall |
WinkerDu
force-pushed
the
master-initial-root-pswd
branch
from
f5fc04d to
b646ce1
Compare
Contributor
Author
|
run buildall |
Contributor
Author
|
run buildall |
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
github-actions
bot
added
the
approved
Contributor
|
PR approved by at least one committer and no changes requested. |
Contributor
|
PR approved by anyone and no changes requested. |
Contributor
|
Maybe documentation should be provided on how to convert a plaintext password to |
github-actions
bot
removed
the
approved
Contributor
Author
|
run buildall |
Contributor
Author
@luozenglin new commit is pushed, PTAL |
github-actions
bot
added
the
approved
Contributor
|
PR approved by at least one committer and no changes requested. |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
morningman
reviewed
Nov 26, 2023
| Default: empty string | ||
| s it possible to dynamically configure: false |
Contributor
There was a problem hiding this comment.
Suggested change
| s it possible to dynamically configure: false | |
| Is it possible to dynamically configure: false |
| public static double diagnose_balance_max_tablet_num_ratio = 1.1; | ||
|
|
||
| @ConfField(masterOnly = true, description = { | ||
| "设置 root 用户初始化2阶段 SHA-1 加密密码，默认为''，即不设置 root 密码。" |
Contributor
There was a problem hiding this comment.
可以多加一句，说明下如何获取 SHA-1 加密后的密码值
WinkerDu
dismissed stale reviews from chenlinzhong and luozenglin
via
9c1bddf
Contributor
Author
|
run buildall |
Contributor
Author
|
run buildall |
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
Contributor
Author
|
run feut |
WinkerDu
added a commit
to WinkerDu/doris
that referenced
this pull request
Nov 27, 2023
seawinde
pushed a commit
to seawinde/doris
that referenced
this pull request
Nov 28, 2023
luozenglin
pushed a commit
that referenced
this pull request
Nov 28, 2023
WinkerDu
added a commit
to WinkerDu/doris
that referenced
this pull request
Nov 29, 2023
eldenmoon
pushed a commit
to eldenmoon/incubator-doris
that referenced
this pull request
Dec 3, 2023
eldenmoon
pushed a commit
to eldenmoon/incubator-doris
that referenced
this pull request
Dec 3, 2023
eldenmoon
added a commit
that referenced
this pull request
Dec 4, 2023
* [chore](case) Use correct insert stmt for cold heat separation case #27546 (#27585) Co-authored-by: AlexYue <yj976240184@gmail.com> * [enhance](S3) Print the error detail for every s3 operation (#27572) (#27615) * [nereids] fix stats error when using dateTime type filter #27571 (#27577) * [fix](planner)sort node should materialized required slots for itself #27605 (#27620) * [fix](Nereids) non-deterministic expression should not be constant (#27606) (#27631) * [enhancement](stats) Add process for aggstate type #27640 (#27642) * [Fix](statistics)Fix bug and improve auto analyze. (#27626) (#27657) 1. Implement needReAnalyzeTable for ExternalTable. For now, external table will not be reanalyzed in 10 days. 2. For HiveMetastoreCache.loadPartitions, handle the empty iterator case to avoid Index out of boundary exception. 3. Wrap handle show analyze loop with try catch, so that when one table failed (for example, catalog dropped so the table couldn't be found anymore), we can still show the other tables. 4. For now, only OlapTable and Hive HMSExternalTable support sample analyze, throw exception for other types of table. 5. In StatisticsCollector, call constructJob after createTableLevelTaskForExternalTable to avoid NPE. * [profile](bugfix) should not cache profile content because the profile may not be a full profile (#27635) --------- Co-authored-by: yiguolei <yiguolei@gmail.com> * [Enhance](fe) Support setting initial root password when FE firstly launch (#27438) (#27603) * [opt](plan) only lock olap table when query plan #27639 (#27656) bp #27639 * select coordinator node from user's tag when exec streaming load (#27106) (#27677) * [fix](statistics)Need to recalculate health value when table row count become 0 #27673 (#27674) backport #27673 * [fix](statistics)Fix sample min max npe bug #27702 (#27707) backport #27702 * [Bug](join) try fix wrong _has_null_in_build_side setted (#27684) (#27710) * [Fix](show-load)Show load npe(userinfo is null) (#27698) (#27719) * [pick](nereids)temporary partition is always pruned #27636 (#27722) * [enhancement](stats) limit bq cap size for analyze task #27685 (#27687) * [improvement](statistics) Add config for the threshold of column count for auto analyze #27713 (#27723) * [doc](fix) k8s operator docs fix to 2.0 (#27476) * [Improvement](planner)support select tablets with nereids optimize #23164 #23365 (#27740) #23164 #23365 * [FIX](complextype)fix complex type hash equals (#27743) * [fix](statistics) Fix show auto analyze missing jobs bug (#27761) * [bugfix](topn) fix coredump in copy_column_data_to_block when nullable mismatch return RuntimeError if copy_column_data_to_block nullable mismatch to avoid coredump in input_col_ptr->filter_by_selector(sel_rowid_idx, select_size, raw_res_ptr) . The problem is reported by a doris user but I can not reproduce it, so there is no testcase added currently. * [opt](stats) Use escape rather than base64 for min/max value #27746 (#27748) * [refactor](http) disable snapshot and get_log_file api (#27724) (#27770) * [branch-2.0](pick 27738) Warning log to trace send fragment #27738 (#27760) * [branch-2.0](pick #27771) Add more detail msg for waitRPC exception (#27773) * [Bug](pipeline) prevent PipelineFragmentContext destruct early (#27790) * [deps](compression) Opt gzip decompress by libdeflate on X86 and X86_64 platforms: 1. Add libdeflate lib. (#27542) (#27711) Backport from #27542. * [FIX](case)fix case truncate table first #27792 * [doc](stats) add auto_analyze_table_width_threshold description. (#27818) (#27832) * [fix](bdbje) Fix bdbje logging level not work (#27597) (#27788) * `EnvironmentConfig.FILE_LOGGING_LEVEL` only set FileHandlerLevel, we should set logger level firstly, otherwise it will not take effect. * [Opt](compression) Opt gzip decompress by libdeflate on X86 and X86_64 platforms: 2. Opt gzip decompression by libdeflate lib. (#27669) (#27801) Backport from #27669. * [branch-2.0](fix) Fix broken exception message #27836 * [Bug](func) coredump in equal for null in function (#27843) * [minor](stats) Update olap table row count after analyze (#27858) pick from master #27814 * [fix](stats)min and max return NaN when table is empty (#27863) fix analyze empty table and min/max null value bug: 1. Skip empty analyze task for sample analyze task. (Full analyze task already skipped). 2. Check sample rows is not 0 before calculate the scale factor. 3. Remove ' in sql template after remove base64 encoding for min/max value. backport #27862 * [minor](stats) Throw error when sync analyze failed (#27846) pick from master #27845 * [fix](stats) Don't save colToPartitions anymore to save mem (#27880) pick from master #27879 * [fix](nereids) set operation's result type is wrong if decimal overflows (#27872) pick from master #27870 * [Config] Modify the default value of tablet_schema_cache_recycle_interval (#27877) * [fix](like_func) incorrect result of like with 'NO_BACKSLASH_ESCAPES' mode(#27842) (#27851) * [fix](fe) Fix show frontends npt in some situations (#27295) (#27789) ``` java.lang.NullPointerException: null at com.sleepycat.je.rep.util.ReplicationGroupAdmin.getMasterSocket(ReplicationGroupAdmin.java:191) at com.sleepycat.je.rep.util.ReplicationGroupAdmin.doMessageExchange(ReplicationGroupAdmin.java:607) at com.sleepycat.je.rep.util.ReplicationGroupAdmin.getGroup(ReplicationGroupAdmin.java:406) at org.apache.doris.ha.BDBHA.getElectableNodes(BDBHA.java:132) at org.apache.doris.common.proc.FrontendsProcNode.getFrontendsInfo(FrontendsProcNode.java:84) at org.apache.doris.qe.ShowExecutor.handleShowFrontends(ShowExecutor.java:1923) at org.apache.doris.qe.ShowExecutor.execute(ShowExecutor.java:355) at org.apache.doris.qe.StmtExecutor.handleShow(StmtExecutor.java:2113) ... ``` * [branch-2.0](fix) Fix extremely high CPU usage caused by rf merge #27894 (#27895) * [fix](stacktrace) ignore stacktrace for error code INVALID_ARGUMENT INVERTED_INDEX_NOT_IMPLEMENTED (#27898) * ignore stacktrace for error INVALID_ARGUMENT INVERTED_INDEX_NOT_IMPLEMENTED * AndBlockColumnPredicate::evaluate * [opt](nereids) Branch-2.0: remove partition & histogram from col stats to reduce memory usage #27885 (#27896) * [pick](Nereids) temporary partition is selected only if user manually specified: Branch-2.0 #27893 (#27905) * [fix](multi-catalog)support the max compute partition prune (#27154) (#27902) backport #27154 * [fix](Nereids) should not push down project to the nullable side of outer join #27912 (#27913) * fix compile --------- Co-authored-by: Dongyang Li <hello_stephen@qq.com> Co-authored-by: AlexYue <yj976240184@gmail.com> Co-authored-by: xzj7019 <131111794+xzj7019@users.noreply.github.com> Co-authored-by: starocean999 <40539150+starocean999@users.noreply.github.com> Co-authored-by: morrySnow <101034200+morrySnow@users.noreply.github.com> Co-authored-by: AKIRA <33112463+Kikyou1997@users.noreply.github.com> Co-authored-by: Jibing-Li <64681310+Jibing-Li@users.noreply.github.com> Co-authored-by: yiguolei <676222867@qq.com> Co-authored-by: yiguolei <yiguolei@gmail.com> Co-authored-by: DuRipeng <453243496@qq.com> Co-authored-by: Mingyu Chen <morningman@163.com> Co-authored-by: wangbo <wangbo@apache.org> Co-authored-by: Pxl <pxl290@qq.com> Co-authored-by: Calvin Kirs <acm_master@163.com> Co-authored-by: minghong <englefly@gmail.com> Co-authored-by: catpineapple <42031973+catpineapple@users.noreply.github.com> Co-authored-by: amory <wangqiannan@selectdb.com> Co-authored-by: Kang <kxiao.tiger@gmail.com> Co-authored-by: zhiqiang <seuhezhiqiang@163.com> Co-authored-by: Qi Chen <kaka11.chen@gmail.com> Co-authored-by: Lei Zhang <27994433+SWJTU-ZhangLei@users.noreply.github.com> Co-authored-by: HappenLee <happenlee@hotmail.com> Co-authored-by: Lightman <31928846+Lchangliang@users.noreply.github.com> Co-authored-by: Jerry Hu <mrhhsg@gmail.com> Co-authored-by: slothever <18522955+wsjz@users.noreply.github.com>
gnehil
pushed a commit
to gnehil/doris
that referenced
this pull request
Dec 4, 2023
XuJianxu
pushed a commit
to XuJianxu/doris
that referenced
this pull request
Dec 14, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Issue Number: close #xxx
Doris FE firstly launch with no root password for now. In my organization scene, we use Doris operator start a Doris cluster, then
set passwordfor root user via peripheral system, which increase system Complexity.This pr introduces FE config
initial_root_passwordto set root user initial 2-staged SHA-1 encrypted password, default as '', means no root password. Subsequent 'set password' operations for root user will overwrite the initial root password.if
initial_root_passwordis not a valid 2-staged SHA-1 encrypted password, no root password set as before.Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...