Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18067 )
Change subject: [ranger] clarify error message from subprocess ...................................................................... [ranger] clarify error message from subprocess Often times, we'll see error messages like the following: org.apache.kudu.client.NonRecoverableException: unable to send message: Other end of pipe was closed at org.apache.kudu.client.KuduException.transformException(KuduException.java:110) at org.apache.kudu.client.KuduClient.joinAndHandleException(KuduClient.java:470) at org.apache.kudu.client.KuduClient.openTable(KuduClient.java:288) ... with litle explanation of what pipe was closed, when really, it's always the Ranger subprocess pipe. This patch adds that context so it's clear to users that their pipe-related issues are Ranger client issues. I tested this manually by running a Ranger test, killing the Ranger client subprocess, and witnessing the following Kudu C++ client error message: End of file: Error creating table db.table on the master: Failed to execute Ranger client subprocess request: unable to send message: Other end of pipe was closed Change-Id: I4a0fcd899439a822c6239494fedb0895adc046fd Reviewed-on: http://gerrit.cloudera.org:8080/18067 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/ranger/ranger_client.cc M src/kudu/subprocess/subprocess_proxy.h 2 files changed, 13 insertions(+), 6 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I4a0fcd899439a822c6239494fedb0895adc046fd Gerrit-Change-Number: 18067 Gerrit-PatchSet: 4 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)
