You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Sailesh Mukil (Code Review)" <ge...@cloudera.org> on 2016/03/08 03:24:38 UTC

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-2599: Pseudo-random sleep before acquiring kerberos ticket possibly not really pseudo-random.

Hello Internal Jenkins, Silvius Rus,

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

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

to review the following change.

Change subject: IMPALA-2599: Pseudo-random sleep before acquiring kerberos ticket possibly not really pseudo-random.
......................................................................

IMPALA-2599: Pseudo-random sleep before acquiring kerberos ticket possibly not really pseudo-random.

A pseudo-random number generator was used to add a sleep before
acquiring a kerberos ticket so that there is minimal traffic to the
KDC. This generator was not seeded which resulted in every impalad
waking up at the same time to contact the KDC for a ticket.

This change seeds the generator with a std::random_device so that
different impalads have different sleep times.

Change-Id: I727a1b68aca43546bb79603b0ca83163ee169aa9
Reviewed-on: http://gerrit.cloudera.org:8080/2429
Reviewed-by: Sailesh Mukil <sa...@cloudera.com>
Reviewed-by: Silvius Rus <sr...@cloudera.com>
Tested-by: Internal Jenkins
---
M be/src/rpc/authentication.cc
1 file changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/76/2476/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2476
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I727a1b68aca43546bb79603b0ca83163ee169aa9
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.5.0_5.7.0
Gerrit-Owner: Sailesh Mukil <sa...@cloudera.com>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Silvius Rus <sr...@cloudera.com>