Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 87FA5182AB for ; Mon, 30 Nov 2015 07:49:17 +0000 (UTC) Received: (qmail 42306 invoked by uid 500); 30 Nov 2015 07:49:17 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 42219 invoked by uid 500); 30 Nov 2015 07:49:17 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 42201 invoked by uid 99); 30 Nov 2015 07:49:16 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 30 Nov 2015 07:49:16 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 9FDE01DDC3F; Mon, 30 Nov 2015 07:49:14 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8278994842852776540==" MIME-Version: 1.0 Subject: Re: Review Request 40315: HIVE-12341 LLAP security From: "Lefty Leverenz" To: "Siddharth Seth" , "Gopal V" Cc: "Lefty Leverenz" , "hive" , "Sergey Shelukhin" Date: Mon, 30 Nov 2015 07:49:14 -0000 Message-ID: <20151130074914.28068.80832@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Lefty Leverenz" X-ReviewGroup: hive X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/40315/ X-Sender: "Lefty Leverenz" References: <20151126013823.24387.66196@reviews.apache.org> In-Reply-To: <20151126013823.24387.66196@reviews.apache.org> Reply-To: "Lefty Leverenz" X-ReviewRequest-Repository: hive-git --===============8278994842852776540== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40315/#review108296 ----------------------------------------------------------- common/src/java/org/apache/hadoop/hive/conf/HiveConf.java (lines 2357 - 2358) Spell out ZooKeeper in the description. common/src/java/org/apache/hadoop/hive/conf/HiveConf.java (lines 2359 - 2361) Spell out ZooKeeper in the description. common/src/java/org/apache/hadoop/hive/conf/HiveConf.java (lines 2362 - 2363) Spell out ZooKeeper in the description. common/src/java/org/apache/hadoop/hive/conf/HiveConf.java (lines 2367 - 2368) Should this have a TimeValidator (like the next two Spark parameters, lines 2372-2376)? Also, the default is 14 * 24 * 3600 but the comment says DelegationTokenManager default is 1 week instead of 2 weeks, so is DelegationTokenManager something different from this parameter? - Lefty Leverenz On Nov. 26, 2015, 1:38 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40315/ > ----------------------------------------------------------- > > (Updated Nov. 26, 2015, 1:38 a.m.) > > > Review request for hive, Gopal V and Siddharth Seth. > > > Repository: hive-git > > > Description > ------- > > see JIRA > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java db942b0 > llap-client/src/java/org/apache/hadoop/hive/llap/io/api/LlapIoProxy.java 4c31e32 > llap-client/src/java/org/apache/hadoop/hive/llap/io/api/LlapProxy.java PRE-CREATION > llap-client/src/java/org/apache/hadoop/hive/llap/security/LlapTokenIdentifier.java PRE-CREATION > llap-client/src/java/org/apache/hadoop/hive/llap/security/LlapTokenProvider.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/LlapDaemonProtocolBlockingPB.java 5ad2344 > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/LlapManagementProtocolBlockingPB.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/LlapDaemon.java 98b1ccd > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/LlapDaemonProtocolClientImpl.java 4b13277 > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/LlapDaemonProtocolServerImpl.java 784c631 > llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/LlapManagementProtocolClientImpl.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/protocol/LlapTaskUmbilicalProtocol.java fae7654 > llap-server/src/java/org/apache/hadoop/hive/llap/security/LlapDaemonPolicyProvider.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/security/LlapSecurityHelper.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/security/LlapServerSecurityInfo.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/security/LlapTokenSelector.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/security/SecretManager.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskCommunicator.java d327fc0 > llap-server/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapUmbilicalPolicyProvider.java PRE-CREATION > llap-server/src/java/org/apache/hadoop/hive/llap/tezplugins/TaskCommunicator.java 33e998c > llap-server/src/main/resources/META-INF/services/org.apache.hadoop.security.SecurityInfo PRE-CREATION > llap-server/src/main/resources/META-INF/services/org.apache.hadoop.security.token.TokenRenewer PRE-CREATION > llap-server/src/protobuf/LlapDaemonProtocol.proto 0ba6acf > llap-server/src/test/org/apache/hadoop/hive/llap/daemon/MiniLlapCluster.java 4525ab9 > llap-server/src/test/org/apache/hadoop/hive/llap/daemon/impl/TestLlapDaemonProtocolServerImpl.java 8d45c95 > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java a210b95 > ql/src/java/org/apache/hadoop/hive/ql/exec/GlobalWorkMapFactory.java 59ee347 > ql/src/java/org/apache/hadoop/hive/ql/exec/ObjectCacheFactory.java 3d9771a > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/MapRecordProcessor.java 914b4e7 > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/ReduceRecordProcessor.java efcf88c > ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 07f26be > ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java 3feab1a > serde/src/java/org/apache/hadoop/hive/serde2/AbstractSerDe.java 9269ff4 > > Diff: https://reviews.apache.org/r/40315/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > > --===============8278994842852776540==--