From reviews-return-107-archive-asf-public=cust-asf.ponee.io@livy.incubator.apache.org Thu Nov 29 01:57:18 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 5CDB118077A for ; Thu, 29 Nov 2018 01:57:18 +0100 (CET) Received: (qmail 12899 invoked by uid 500); 29 Nov 2018 00:57:17 -0000 Mailing-List: contact reviews-help@livy.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@livy.incubator.apache.org Delivered-To: mailing list reviews@livy.incubator.apache.org Received: (qmail 12745 invoked by uid 99); 29 Nov 2018 00:57:16 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 29 Nov 2018 00:57:16 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 1A193E0A2F; Thu, 29 Nov 2018 00:57:16 +0000 (UTC) From: vanzin To: reviews@livy.apache.org Reply-To: reviews@livy.apache.org References: In-Reply-To: Subject: [GitHub] incubator-livy pull request #117: [LIVY-502] Remove dependency on hive-exec Content-Type: text/plain Message-Id: <20181129005716.1A193E0A2F@git1-us-west.apache.org> Date: Thu, 29 Nov 2018 00:57:16 +0000 (UTC) Github user vanzin commented on a diff in the pull request: https://github.com/apache/incubator-livy/pull/117#discussion_r237314382 --- Diff: server/src/main/scala/org/apache/livy/LivyConf.scala --- @@ -98,10 +98,78 @@ object LivyConf { val LAUNCH_KERBEROS_REFRESH_INTERVAL = Entry("livy.server.launch.kerberos.refresh-interval", "1h") val KINIT_FAIL_THRESHOLD = Entry("livy.server.launch.kerberos.kinit-fail-threshold", 5) + // Thrift configurations val THRIFT_SERVER_ENABLED = Entry("livy.server.thrift.enabled", false) val THRIFT_INCR_COLLECT_ENABLED = Entry("livy.server.thrift.incrementalCollect", false) val THRIFT_SESSION_CREATION_TIMEOUT = Entry("livy.server.thrift.session.creationTimeout", "10m") val THRIFT_SERVER_JAR_LOCATION = Entry("livy.server.thrift.jarLocation", null) + // The following configs are the same present in Hive + val THRIFT_RESULTSET_DEFAULT_FETCH_SIZE = + Entry("livy.server.thrift.resultset.default.fetch.size", 1000) + val THRIFT_SPNEGO_PRINCIPAL = Entry("livy.server.thrift.authentication.spnego.principal", "") --- End diff -- Won't this be exactly the same as `AUTH_KERBEROS_PRINCIPAL`? The spnego principal must be `HTTP`, so it's not like they can be different. Which also means the keytab config will have to match, too. ---