XiaokaiWang has posted comments on this change. ( http://gerrit.cloudera.org:8080/13796 )
Change subject: KUDU-2722 (table level): Support new 'write_enabled' table extra config ...................................................................... Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/13796/2/src/kudu/tserver/tablet_service.cc File src/kudu/tserver/tablet_service.cc: http://gerrit.cloudera.org:8080/#/c/13796/2/src/kudu/tserver/tablet_service.cc@1084 PS2, Line 1084: boost::optional<TableExtraConfigPB> extra_config = tablet->metadata()->extra_config(); > Still missed this. Sorry, done now. http://gerrit.cloudera.org:8080/#/c/13796/7/src/kudu/tserver/tablet_service.cc File src/kudu/tserver/tablet_service.cc: http://gerrit.cloudera.org:8080/#/c/13796/7/src/kudu/tserver/tablet_service.cc@1146 PS7, Line 1146: Status s = Status::InvalidArgument("rejecting write request: tablet is not writable"); : SetupErrorAndRespond(resp->mutable_error(), s, > Nit: why not combine these as before? The line length is long, it's more than 100 characters after indent. -- To view, visit http://gerrit.cloudera.org:8080/13796 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibd3768eda36d9574be9c41e7cd3dd81cc5ae8f3a Gerrit-Change-Number: 13796 Gerrit-PatchSet: 7 Gerrit-Owner: XiaokaiWang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: XiaokaiWang <[email protected]> Gerrit-Reviewer: Yao Xu <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Thu, 11 Jul 2019 13:39:45 +0000 Gerrit-HasComments: Yes
