Hello Alexey Serbin, Attila Bukor,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/18067

to review the following change.


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 little 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 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
---
M src/kudu/ranger/ranger_client.cc
M src/kudu/subprocess/subprocess_proxy.h
2 files changed, 13 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/18067/1
--
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: newchange
Gerrit-Change-Id: I4a0fcd899439a822c6239494fedb0895adc046fd
Gerrit-Change-Number: 18067
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>

Reply via email to