You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by kg...@apache.org on 2020/02/04 16:23:47 UTC

[hive] branch master updated (06c5923 -> a21ca0e5)

This is an automated email from the ASF dual-hosted git repository.

kgyrtkirk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git.


    from 06c5923  HIVE-21215: Read Parquet INT64 timestamp (Marta Kuczora, reviewed by Karen Coppage and Peter Vary)
     new 7abc5f7  HIVE-22780: Upgrade slf4j version to 1.7.30 (David Lavati via Miklos Gergely)
     new 684d302  HIVE-22801: Debug log is flooded with some debug dump stack (Zoltan Haindrich reviewed by Miklos Gergely)
     new a21ca0e5 HIVE-22680: Replace Base64 in druid-handler Package (David Mollitor via Zoltan Haindrich)

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java    | 6 ++----
 itests/qtest-druid/pom.xml                                          | 2 +-
 itests/qtest/pom.xml                                                | 2 +-
 kafka-handler/pom.xml                                               | 2 +-
 pom.xml                                                             | 2 +-
 .../src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java | 6 +++++-
 standalone-metastore/metastore-tools/pom.xml                        | 2 +-
 storage-api/pom.xml                                                 | 2 +-
 testutils/ptest2/pom.xml                                            | 2 +-
 9 files changed, 14 insertions(+), 12 deletions(-)


[hive] 02/03: HIVE-22801: Debug log is flooded with some debug dump stack (Zoltan Haindrich reviewed by Miklos Gergely)

Posted by kg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

kgyrtkirk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git

commit 684d302d196dfde624d727c4d8270e2bc0ed4ca5
Author: Zoltan Haindrich <ki...@rxd.hu>
AuthorDate: Tue Feb 4 14:08:10 2020 +0000

    HIVE-22801: Debug log is flooded with some debug dump stack (Zoltan Haindrich reviewed by Miklos Gergely)
    
    Signed-off-by: Zoltan Haindrich <ki...@rxd.hu>
---
 .../src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
index 80d2111..793d041 100644
--- a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
+++ b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
@@ -9682,7 +9682,11 @@ public class ObjectStore implements RawStore, Configurable {
 
   private void debugLog(final String message) {
     if (LOG.isDebugEnabled()) {
-      LOG.debug("{}", message, new Exception("Debug Dump Stack Trace (Not an Exception)"));
+      if (LOG.isTraceEnabled()) {
+        LOG.debug("{}", message, new Exception("Debug Dump Stack Trace (Not an Exception)"));
+      } else {
+        LOG.debug("{}", message);
+      }
     }
   }
 


[hive] 01/03: HIVE-22780: Upgrade slf4j version to 1.7.30 (David Lavati via Miklos Gergely)

Posted by kg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

kgyrtkirk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git

commit 7abc5f71d56628064caf16a22f72c6674ef40df2
Author: David Lavati <da...@gmail.com>
AuthorDate: Tue Feb 4 14:04:32 2020 +0000

    HIVE-22780: Upgrade slf4j version to 1.7.30 (David Lavati via Miklos Gergely)
    
    Signed-off-by: Zoltan Haindrich <ki...@rxd.hu>
---
 itests/qtest-druid/pom.xml                   | 2 +-
 itests/qtest/pom.xml                         | 2 +-
 kafka-handler/pom.xml                        | 2 +-
 pom.xml                                      | 2 +-
 standalone-metastore/metastore-tools/pom.xml | 2 +-
 storage-api/pom.xml                          | 2 +-
 testutils/ptest2/pom.xml                     | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/itests/qtest-druid/pom.xml b/itests/qtest-druid/pom.xml
index 05692c7..6da7273 100644
--- a/itests/qtest-druid/pom.xml
+++ b/itests/qtest-druid/pom.xml
@@ -44,7 +44,7 @@
     <druid.guava.version>16.0.1</druid.guava.version>
     <druid.guice.version>4.1.0</druid.guice.version>
     <kafka.test.version>2.0.0</kafka.test.version>
-    <slf4j.version>1.7.25</slf4j.version>
+    <slf4j.version>1.7.30</slf4j.version>
   </properties>
       <dependencies>
         <dependency>
diff --git a/itests/qtest/pom.xml b/itests/qtest/pom.xml
index be8e377..f6fce77 100644
--- a/itests/qtest/pom.xml
+++ b/itests/qtest/pom.xml
@@ -39,7 +39,7 @@
     <clustermode></clustermode>
     <execute.beeline.tests>false</execute.beeline.tests>
     <test.dfs.mkdir>-mkdir -p</test.dfs.mkdir>
-    <slf4j.version>1.7.25</slf4j.version>
+    <slf4j.version>1.7.30</slf4j.version>
   </properties>
 
   <dependencies>
diff --git a/kafka-handler/pom.xml b/kafka-handler/pom.xml
index a66a70a..6ad41de 100644
--- a/kafka-handler/pom.xml
+++ b/kafka-handler/pom.xml
@@ -115,7 +115,7 @@
     <dependency>
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
-      <version>1.7.25</version>
+      <version>1.7.30</version>
       <scope>test</scope>
     </dependency>
   </dependencies>
diff --git a/pom.xml b/pom.xml
index 2dd2128..2947a29 100644
--- a/pom.xml
+++ b/pom.xml
@@ -203,7 +203,7 @@
     <plexus.version>1.5.6</plexus.version>
     <protobuf.version>2.5.0</protobuf.version>
     <stax.version>1.0.1</stax.version>
-    <slf4j.version>1.7.10</slf4j.version>
+    <slf4j.version>1.7.30</slf4j.version>
     <ST4.version>4.0.4</ST4.version>
     <storage-api.version>2.7.0-SNAPSHOT</storage-api.version>
     <tez.version>0.9.1</tez.version>
diff --git a/standalone-metastore/metastore-tools/pom.xml b/standalone-metastore/metastore-tools/pom.xml
index 63f2369..d8c4788 100644
--- a/standalone-metastore/metastore-tools/pom.xml
+++ b/standalone-metastore/metastore-tools/pom.xml
@@ -103,7 +103,7 @@
       <dependency>
         <groupId>org.slf4j</groupId>
         <artifactId>slf4j-log4j12</artifactId>
-        <version>1.7.25</version>
+	<version>1.7.30</version>
       </dependency>
       <!-- https://mvnrepository.com/artifact/org.jetbrains/annotations -->
       <dependency>
diff --git a/storage-api/pom.xml b/storage-api/pom.xml
index 61fdaf0..39b23b0 100644
--- a/storage-api/pom.xml
+++ b/storage-api/pom.xml
@@ -34,7 +34,7 @@
     <guava.version>19.0</guava.version>
     <hadoop.version>3.1.0</hadoop.version>
     <junit.version>4.11</junit.version>
-    <slf4j.version>1.7.10</slf4j.version>
+    <slf4j.version>1.7.30</slf4j.version>
     <maven.checkstyle.plugin.version>2.17</maven.checkstyle.plugin.version>
     <checkstyle.conf.dir>${basedir}/checkstyle/</checkstyle.conf.dir>
   </properties>
diff --git a/testutils/ptest2/pom.xml b/testutils/ptest2/pom.xml
index 6d43056..cc04607 100644
--- a/testutils/ptest2/pom.xml
+++ b/testutils/ptest2/pom.xml
@@ -133,7 +133,7 @@ limitations under the License.
     <dependency>
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
-      <version>1.7.10</version>
+      <version>1.7.30</version>
     </dependency>
     <dependency>
       <groupId>org.springframework</groupId>


[hive] 03/03: HIVE-22680: Replace Base64 in druid-handler Package (David Mollitor via Zoltan Haindrich)

Posted by kg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

kgyrtkirk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git

commit a21ca0e59967bff5570288fcc5b5b7e666e0e89a
Author: David Mollitor <da...@gmail.com>
AuthorDate: Tue Feb 4 14:17:15 2020 +0000

    HIVE-22680: Replace Base64 in druid-handler Package (David Mollitor via Zoltan Haindrich)
    
    Signed-off-by: Zoltan Haindrich <ki...@rxd.hu>
---
 .../org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java    | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/druid-handler/src/java/org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java b/druid-handler/src/java/org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java
index 8e10cd7..12603c1 100644
--- a/druid-handler/src/java/org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java
+++ b/druid-handler/src/java/org/apache/hadoop/hive/druid/security/DruidKerberosUtil.java
@@ -18,7 +18,6 @@
 
 package org.apache.hadoop.hive.druid.security;
 
-import org.apache.commons.codec.binary.Base64;
 import org.apache.hadoop.security.authentication.client.AuthenticatedURL;
 import org.apache.hadoop.security.authentication.client.AuthenticationException;
 import org.apache.hadoop.security.authentication.util.KerberosUtil;
@@ -33,7 +32,7 @@ import org.slf4j.LoggerFactory;
 import java.net.CookieStore;
 import java.net.HttpCookie;
 import java.net.URI;
-import java.nio.charset.StandardCharsets;
+import java.util.Base64;
 import java.util.List;
 import java.util.concurrent.locks.ReentrantLock;
 
@@ -42,7 +41,6 @@ import java.util.concurrent.locks.ReentrantLock;
  */
 public final class DruidKerberosUtil {
   protected static final Logger LOG = LoggerFactory.getLogger(DruidKerberosUtil.class);
-  private static final Base64 BASE_64_CODEC = new Base64(0);
   private static final ReentrantLock KERBEROS_LOCK = new ReentrantLock(true);
 
   private DruidKerberosUtil() {
@@ -78,7 +76,7 @@ public final class DruidKerberosUtil {
       gssContext.dispose();
       // Base64 encoded and stringified token for server
       LOG.debug("Got valid challenge for host {}", serverName);
-      return new String(BASE_64_CODEC.encode(outToken), StandardCharsets.US_ASCII);
+      return Base64.getEncoder().encodeToString(outToken);
     } catch (GSSException | IllegalAccessException | NoSuchFieldException | ClassNotFoundException e) {
       throw new AuthenticationException(e);
     } finally {