Adding support for url encoding/decoding

jasperjiaguo · jasperjiaguo · commit 6dcee76823d7 · 2022-03-21T15:56:35.000-07:00
diff --git a/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java b/pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java
@@ -456,7 +456,7 @@ public static int strcmp(String input1, String input2) {
   @ScalarFunction
   public static String encodeUrl(String input)
       throws UnsupportedEncodingException {
-      return URLEncoder.encode(input, StandardCharsets.UTF_8);
+      return URLEncoder.encode(input, StandardCharsets.UTF_8.toString());
   }
 
   /**
@@ -468,6 +468,6 @@ public static String encodeUrl(String input)
   @ScalarFunction
   public static String decodeUrl(String input)
       throws UnsupportedEncodingException {
-    return URLDecoder.decode(input, StandardCharsets.UTF_8);
+    return URLDecoder.decode(input, StandardCharsets.UTF_8.toString());
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -456,7 +456,7 @@ public static int strcmp(String input1, String input2) {`
`456`	`456`	`@ScalarFunction`
`457`	`457`	`public static String encodeUrl(String input)`
`458`	`458`	`throws UnsupportedEncodingException {`
`459`		`- return URLEncoder.encode(input, StandardCharsets.UTF_8);`
	`459`	`+ return URLEncoder.encode(input, StandardCharsets.UTF_8.toString());`
`460`	`460`	`}`
`461`	`461`
`462`	`462`	`/**`
`@@ -468,6 +468,6 @@ public static String encodeUrl(String input)`
`468`	`468`	`@ScalarFunction`
`469`	`469`	`public static String decodeUrl(String input)`
`470`	`470`	`throws UnsupportedEncodingException {`
`471`		`- return URLDecoder.decode(input, StandardCharsets.UTF_8);`
	`471`	`+ return URLDecoder.decode(input, StandardCharsets.UTF_8.toString());`
`472`	`472`	`}`
`473`	`473`	`}`