fix(array functions): Add support to array_has_duplicates for input of type json (facebookincubator#12158)

Summary:

Minor update to function array_has_duplicates which registers json as input in array registration file. No need to cast input as json is already treated as a varchar.

Reviewed By: kgpai

Differential Revision: D68580104

Author: peterenescu

velox/functions/prestosql/registration/ArrayFunctionsRegistration.cpp

@@ -298,6 +298,7 @@ void registerArrayFunctions(const std::string& prefix) {
   registerArrayHasDuplicatesFunctions<int64_t>(prefix);
   registerArrayHasDuplicatesFunctions<int128_t>(prefix);
   registerArrayHasDuplicatesFunctions<Varchar>(prefix);
+  registerArrayHasDuplicatesFunctions<Json>(prefix);
 
   registerArrayFrequencyFunctions<bool>(prefix);
   registerArrayFrequencyFunctions<int8_t>(prefix);

velox/functions/prestosql/tests/ArrayHasDuplicatesTest.cpp

@@ -16,6 +16,7 @@
 
 #include <optional>
 #include "velox/functions/prestosql/tests/utils/FunctionBaseTest.h"
+#include "velox/functions/prestosql/types/JsonType.h"
 
 using namespace facebook::velox;
 using namespace facebook::velox::test;
@@ -145,3 +146,42 @@ TEST_F(ArrayHasDuplicatesTest, nullFreeStrings) {
   auto expected = makeFlatVector<bool>({false, true, false, true});
   testExpr(expected, "array_has_duplicates(C0)", {array});
 }
+
+TEST_F(ArrayHasDuplicatesTest, json) {
+  auto result = evaluate(
+      "array_has_duplicates(C0)",
+      makeRowVector({makeNullableArrayVector<StringView>(
+          {{R"({"key":"value"})", R"({"key":"value"})"}}, ARRAY(JSON()))}));
+  assertEqualVectors(makeFlatVector<bool>(true), result);
+
+  result = evaluate(
+      "array_has_duplicates(C0)",
+      makeRowVector({makeNullableArrayVector<StringView>(
+          {{R"({"key":"value"})", R"({"key":"another_value"})"},
+           {R"({"key":"value"})"},
+           {R"({"key":"same_value"})",
+            R"({"key":"another_value"})",
+            R"({"key":"same_value"})"},
+           {std::nullopt, std::nullopt},
+           {R"({"key":"value"})",
+            R"({"key":"another_value"})",
+            R"({"another_key":"value"})"},
+           {R"({"key": "value\with\backslash"})",
+            R"({"key": "value\with\backslash"})"},
+           {R"({"key": "value\nwith\nnewline"})",
+            R"({"key": "value\nwith\nnewline"})"},
+           {R"({"key": "value with \u00A9 and \u20AC"})",
+            R"({"key": "value with \u00A9 and \u20AC"})"},
+           {R"({"key": "!@#$%^&*()_+-={}:<>?,./~`"})",
+            R"({"key": "!@#$%^&*()_+-={}:<>?,./~`"})"},
+           {R"({"key":"value"})",
+            std::nullopt,
+            R"({"key":"another_value"})",
+            R"({"another_key":"value"})",
+            std::nullopt}},
+          ARRAY(JSON()))}));
+  assertEqualVectors(
+      makeFlatVector<bool>(
+          {false, false, true, true, false, true, true, true, true, true}),
+      result);
+}