Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/19237 )
Change subject: [Flag] Check flags consistency when setting a flag ...................................................................... Patch Set 7: (4 comments) > Patch Set 6: Code-Review+1 > > (4 comments) > > Overall looks good to me! Just a few nits to fix wording in comments and > messages. http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/server/generic_service.cc File src/kudu/server/generic_service.cc: http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/server/generic_service.cc@163 PS6, Line 163: "Failed to roll back flag '$0' to previous value '$1 > nit: not sure the advice to set flag manually is actionable; how about Done http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/server/server_base.proto File src/kudu/server/server_base.proto: http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/server/server_base.proto@80 PS6, Line 80: Check all flags for consistency upon setting a new > nit: how about Done http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/tools/tool_action_common.cc File src/kudu/tools/tool_action_common.cc: http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/tools/tool_action_common.cc@104 PS6, Line 104: If true, Kudu server checks all flags for consistency " : "upon setting a flag. In this mode, the server rolls the flag back > nit: how about Done http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/util/flags.h File src/kudu/util/flags.h: http://gerrit.cloudera.org:8080/#/c/19237/6/src/kudu/util/flags.h@60 PS6, Line 60: // Check for unsafe and experimental flags, and run all group flag validators. : // Returns 'true' if all checks pass and no in > nit: how about Done -- To view, visit http://gerrit.cloudera.org:8080/19237 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9e6e37e0f4e72a2cc3d2316248961315f1757ebc Gerrit-Change-Number: 19237 Gerrit-PatchSet: 7 Gerrit-Owner: Wang Xixu <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Comment-Date: Mon, 21 Nov 2022 01:53:29 +0000 Gerrit-HasComments: Yes
