[hive] branch master updated: HIVE-26139: Encode only '#' characters in HBaseStorageHandler authorization URL (Steve Carlin, reviewed by Alessandro Solimando, Stamatis Zampetakis)

[zabetak]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new f94a6efbf5 HIVE-26139: Encode only '#' characters in 
HBaseStorageHandler authorization URL (Steve Carlin, reviewed by Alessandro 
Solimando, Stamatis Zampetakis)
f94a6efbf5 is described below

commit f94a6efbf5a37d82e6160ff816c86f7240f37a70
Author: Steve Carlin <scar...@cloudera.com>
AuthorDate: Tue Apr 12 11:32:25 2022 -0700

    HIVE-26139: Encode only '#' characters in HBaseStorageHandler authorization 
URL (Steve Carlin, reviewed by Alessandro Solimando, Stamatis Zampetakis)
    
    Remove the global encoding of the authorization URL since it has some
    undesirable side effects.
    
    Closes #3206
---
 .../org/apache/hadoop/hive/hbase/HBaseStorageHandler.java     | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git 
a/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java 
b/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
index 03d455f095..b5cecccd49 100644
--- 
a/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
+++ 
b/hbase-handler/src/java/org/apache/hadoop/hive/hbase/HBaseStorageHandler.java
@@ -19,10 +19,8 @@
 package org.apache.hadoop.hive.hbase;
 
 import java.io.IOException;
-import java.io.UnsupportedEncodingException;
 import java.net.URI;
 import java.net.URISyntaxException;
-import java.net.URLEncoder;
 import java.util.ArrayList;
 import java.util.LinkedHashSet;
 import java.util.List;
@@ -310,12 +308,9 @@ public class HBaseStorageHandler extends 
DefaultStorageHandler
     return new URI(URIString);
   }
 
-  private static String encodeString(String rawString) throws 
URISyntaxException {
-    try {
-      return rawString != null ? URLEncoder.encode(rawString, "UTF-8"): null;
-    } catch (UnsupportedEncodingException e) {
-      throw new URISyntaxException(rawString, "Could not URLEncode string");
-    }
+  private static String encodeString(String rawString) {
+    // Only url encode hash code value for now
+    return rawString != null ? rawString.replace("#", "%23") : null;
   }
 
   /**