impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5129: Use KRPC's Kinit code to avoid expensive fork
Date Mon, 09 Oct 2017 19:10:05 GMT
Sailesh Mukil has posted comments on this change. ( )

Change subject: IMPALA-5129: Use KRPC's Kinit code to avoid expensive fork

Patch Set 5:

File be/src/kudu/security/
PS5, Line 460:   //setenv("KRB5CCNAME", "MEMORY:kudu", 1);
             :   //setenv("KRB5_KTNAME", FLAGS_keytab_file.c_str(), 1);
             :   // We commented out the above since we don't want to overwrite our own Impala
             :   // set environment variables.
> Is this function used for testing only ? If so, would it be better to move 
No, this function is what initializes Kerberos in the kudu security library. See L843 in

Unfortunately, there isn't a clean way to make this configurable, unless we pass in strings
to the InitKerberosForServer() function as parameters for these environment variables, which
makes little sense for Kudu to have in their code base.
PS5, Line 469:   //setenv("KRB5RCACHETYPE", "none", 1);
> Same comment.
Same as above.

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9cea56cc6e7412d87f4c2e92399a2f91ea6af6c7
Gerrit-Change-Number: 7938
Gerrit-PatchSet: 5
Gerrit-Owner: Sailesh Mukil <>
Gerrit-Reviewer: Bikramjeet Vig <>
Gerrit-Reviewer: Michael Ho <>
Gerrit-Reviewer: Sailesh Mukil <>
Gerrit-Comment-Date: Mon, 09 Oct 2017 19:10:05 +0000
Gerrit-HasComments: Yes

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message