Return-Path: X-Original-To: apmail-impala-dev-archive@minotaur.apache.org Delivered-To: apmail-impala-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 0020B19754 for ; Tue, 8 Mar 2016 00:21:39 +0000 (UTC) Received: (qmail 2886 invoked by uid 500); 8 Mar 2016 00:21:39 -0000 Delivered-To: apmail-impala-dev-archive@impala.apache.org Received: (qmail 2840 invoked by uid 500); 8 Mar 2016 00:21:39 -0000 Mailing-List: contact dev-help@impala.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@impala.incubator.apache.org Delivered-To: mailing list dev@impala.incubator.apache.org Received: (qmail 2829 invoked by uid 99); 8 Mar 2016 00:21:39 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 08 Mar 2016 00:21:39 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id 10AE21A04D5 for ; Tue, 8 Mar 2016 00:21:39 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.362 X-Spam-Level: X-Spam-Status: No, score=0.362 tagged_above=-999 required=6.31 tests=[RDNS_DYNAMIC=0.363, SPF_PASS=-0.001] autolearn=disabled Received: from mx2-lw-us.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id FDaJHSGYOEJ6 for ; Tue, 8 Mar 2016 00:21:36 +0000 (UTC) Received: from ip-10-146-233-104.ec2.internal (ec2-75-101-130-251.compute-1.amazonaws.com [75.101.130.251]) by mx2-lw-us.apache.org (ASF Mail Server at mx2-lw-us.apache.org) with ESMTPS id 85A355FAE8 for ; Tue, 8 Mar 2016 00:21:36 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by ip-10-146-233-104.ec2.internal (8.14.4/8.14.4) with ESMTP id u280LZ4t025013; Tue, 8 Mar 2016 00:21:35 GMT Message-Id: <201603080021.u280LZ4t025013@ip-10-146-233-104.ec2.internal> Date: Tue, 8 Mar 2016 00:21:35 +0000 From: "Internal Jenkins (Code Review)" To: Sailesh Mukil , impala-cr@cloudera.com, dev@impala.incubator.apache.org X-Gerrit-MessageType: merged Subject: =?UTF-8?Q?[Impala-CR](cdh5-trunk)_IMPALA-2599:_Pseudo-random_sleep_before_acquiring_kerberos_ticket_possibly_not_really_pseudo-random.=0A?= X-Gerrit-Change-Id: I727a1b68aca43546bb79603b0ca83163ee169aa9 X-Gerrit-ChangeURL: X-Gerrit-Commit: 2ad6a328199a23b2597bc40caa4dba17d71c4b74 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.10-rc0 Internal Jenkins has submitted this change and it was merged. 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 Reviewed-by: Silvius Rus Tested-by: Internal Jenkins --- M be/src/rpc/authentication.cc 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Internal Jenkins: Verified Silvius Rus: Looks good to me, approved Sailesh Mukil: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/2429 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I727a1b68aca43546bb79603b0ca83163ee169aa9 Gerrit-PatchSet: 4 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Sailesh Mukil Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Silvius Rus