This is an automated email from the ASF dual-hosted git repository.
srowen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 940626b [SPARK-15095][FOLLOW-UP][SQL] Remove HiveSessionHook related
code from ThriftServer
940626b is described below
commit 940626b72499d8914141c0452bbf5ad689951f28
Author: Yuming Wang <yumw...@ebay.com>
AuthorDate: Tue Mar 5 07:42:25 2019 -0600
[SPARK-15095][FOLLOW-UP][SQL] Remove HiveSessionHook related code from
ThriftServer
## What changes were proposed in this pull request?
https://github.com/apache/spark/pull/12881 removed `HiveSessionHook`. But
there are still some code related to `HiveSessionHook`.
This PR removes all `HiveSessionHook` related code.
## How was this patch tested?
manual tests
Closes #23957 from wangyum/SPARK-15095.
Authored-by: Yuming Wang <yumw...@ebay.com>
Signed-off-by: Sean Owen <sean.o...@databricks.com>
---
.../cli/session/HiveSessionHookContext.java | 46 -------------------
.../cli/session/HiveSessionHookContextImpl.java | 52 ----------------------
2 files changed, 98 deletions(-)
diff --git
a/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContext.java
b/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContext.java
deleted file mode 100644
index c56a107..0000000
---
a/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContext.java
+++ /dev/null
@@ -1,46 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.apache.hive.service.cli.session;
-
-import org.apache.hadoop.hive.conf.HiveConf;
-/**
- * HiveSessionHookContext.
- * Interface passed to the HiveServer2 session hook execution. This enables
- * the hook implementation to access session config, user and session handle
- */
-public interface HiveSessionHookContext {
-
- /**
- * Retrieve session conf
- * @return
- */
- HiveConf getSessionConf();
-
- /**
- * The get the username starting the session
- * @return
- */
- String getSessionUser();
-
- /**
- * Retrieve handle for the session
- * @return
- */
- String getSessionHandle();
-}
diff --git
a/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContextImpl.java
b/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContextImpl.java
deleted file mode 100644
index 1ee4ac8..0000000
---
a/sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionHookContextImpl.java
+++ /dev/null
@@ -1,52 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.apache.hive.service.cli.session;
-
-import org.apache.hadoop.hive.conf.HiveConf;
-
-/**
- *
- * HiveSessionHookContextImpl.
- * Session hook context implementation which is created by session manager
- * and passed to hook invocation.
- */
-public class HiveSessionHookContextImpl implements HiveSessionHookContext {
-
- private final HiveSession hiveSession;
-
- HiveSessionHookContextImpl(HiveSession hiveSession) {
- this.hiveSession = hiveSession;
- }
-
- @Override
- public HiveConf getSessionConf() {
- return hiveSession.getHiveConf();
- }
-
-
- @Override
- public String getSessionUser() {
- return hiveSession.getUserName();
- }
-
- @Override
- public String getSessionHandle() {
- return hiveSession.getSessionHandle().toString();
- }
-}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
