Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/18067 )
Change subject: [ranger] clarify error message from subprocess ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/18067/2/src/kudu/subprocess/subprocess_proxy.h File src/kudu/subprocess/subprocess_proxy.h: http://gerrit.cloudera.org:8080/#/c/18067/2/src/kudu/subprocess/subprocess_proxy.h@95 PS2, Line 95: const std::string subprocess_name_; > nit: might be const char* const subprocess_name_ since the field isn't chan Good point -- I'll instead store a string. -- To view, visit http://gerrit.cloudera.org:8080/18067 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4a0fcd899439a822c6239494fedb0895adc046fd Gerrit-Change-Number: 18067 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Tue, 14 Dec 2021 21:40:32 +0000 Gerrit-HasComments: Yes
