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

@@ -316,6 +316,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,40 @@ TEST_F(ArrayHasDuplicatesTest, nullFreeStrings) {
   auto expected = makeFlatVector<bool>({false, true, false, true});
   testExpr(expected, "array_has_duplicates(C0)", {array});
 }
+
+TEST_F(ArrayHasDuplicatesTest, json) {
+  auto arrayVector = makeNullableArrayVector<StringView>(
+      {{R"({"key":"value"})", R"({"key":"value"})"}}, ARRAY(JSON()));
+  testExpr(
+      makeFlatVector<bool>(true), "array_has_duplicates(C0)", {arrayVector});
+
+  auto test = 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()));
+  testExpr(
+      makeFlatVector<bool>(
+          {false, false, true, true, false, true, true, true, true, true}),
+      "array_has_duplicates(C0)",
+      {test});
+}