Yuqi Du has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18749 )

Change subject: [log] make two line logs more clearly
......................................................................


Patch Set 1:

(1 comment)

Thanks your cr.

http://gerrit.cloudera.org:8080/#/c/18749/1/src/kudu/master/master_main.cc
File src/kudu/master/master_main.cc:

http://gerrit.cloudera.org:8080/#/c/18749/1/src/kudu/master/master_main.cc@36
PS1, Line 36: parse command line flags error,
> Why would anybody be interested to know that it's ParseCommandLineFlags() t
For example, at a vert complex start_tserver.sh, obtain flags infomations and 
start kudu tserver.
(the example is my own special kudu-tserver). At the end the command:

```
bin/kudu-server --disable_core_dumps=true --webserver_port=8070 
--rpc_bind_addresses=0.0.0.0:7070 --superuser_acl=kudu,root,sa_cluster 
--max_clock_sync_error_usec=100000000 --downstream_type=kafka 
--kafka_queue_length=15000 --kafka_target_topic=9 9 kudu_profile_record_stream 
--sync_history_data=false --memory_limit_hard_bytes=27013989990 
--tablet_transaction_memory_limit_mb=128 --sleep_second_per_epoch=0 
--trusted_subnets=0.0.0.0/0 --rpc_authentication=disabled 
--rpc_encryption=disabled 
--tserver_master_addrs=data03:7051,data02:7051,data01:7051 
--log_dir=/metadata/sa_cluster/logs/infinity/ksyncer_kafka 
--fs_data_dirs=/data1/sa_cluster/ksyncer_kafka/ksyncer_data 
--fs_wal_dir=/data1/sa_cluster/ksyncer_kafka/ksyncer_wal 
--kafka_broker_list=data04:9092,data05:9092,data06:9092,data07:9092,data08:9092
usage: bin/kudu-tserver
```
In fact, for some special reason, topic_name is error. But the error output is 
hard to help me, it’s Inconspicuous.
If the usage is more clearly, it can save time to address the problem.



--
To view, visit http://gerrit.cloudera.org:8080/18749
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic4adbcb171473eca4a534ddbb7f438779d859d2d
Gerrit-Change-Number: 18749
Gerrit-PatchSet: 1
Gerrit-Owner: Yuqi Du <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yuqi Du <[email protected]>
Gerrit-Comment-Date: Wed, 20 Jul 2022 03:43:00 +0000
Gerrit-HasComments: Yes

Reply via email to