Alexey Serbin has posted comments on this change.

Change subject: [mini-kdc] /dev/stderr instead of STDERR for kdc logs
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6602/1//COMMIT_MSG
Commit Message:

Line 18: for the error output, but not opening the standard error in read mode.
> Really interesting.  We also do something similar here: https://github.com/
Thanks for pointing to that.  Yes, that's also the case: if running the krb5adm 
in Kudu security tests, the same issue appears.

I found this issue is specific for versions of krb5adm prior to 1.13.  It was 
fixed in changelist a8592307 (dated back to August 2014).  So, it has been 
fixed in newer version of krb5adm binaries, but we have many machines where we 
run RedHat/CentOS 6.x.

I'll update the commit the Java counterpart to use /dev/stderr and the commit 
message to include the information on the fix.


-- 
To view, visit http://gerrit.cloudera.org:8080/6602
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I212e2cceb80acbe88040b16d9055cf01ac8a761b
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to