Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18631 )
Change subject: [rpc] report the names of the unsupported feature flags ...................................................................... [rpc] report the names of the unsupported feature flags While introducing feature flags in the scope of KUDU-2671, I noticed that missing feature flags are not reported in the error messages (at least, I was looking at master_proxy_rpc.cc). This patch addresses that, making corresponding error messages more actionable. As for test coverage, I updated a couple of test scenarios in dynamic_multi_master-test.cc to provide initial coverage. In addition, a follow-up patch will add a new test scenario into flex_partitioning_client-test.cc. Change-Id: Id9805dc3fb4ba0a734ca92198bc5dc6449f588b7 Reviewed-on: http://gerrit.cloudera.org:8080/18631 Tested-by: Kudu Jenkins Reviewed-by: Attila Bukor <[email protected]> --- M src/kudu/client/master_proxy_rpc.cc M src/kudu/client/txn_manager_proxy_rpc.cc M src/kudu/master/dynamic_multi_master-test.cc M src/kudu/tserver/heartbeater.cc 4 files changed, 34 insertions(+), 12 deletions(-) Approvals: Kudu Jenkins: Verified Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18631 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id9805dc3fb4ba0a734ca92198bc5dc6449f588b7 Gerrit-Change-Number: 18631 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
