diff --git a/presto-main/src/main/java/com/facebook/presto/operator/scalar/sql/ArraySqlFunctions.java b/presto-main/src/main/java/com/facebook/presto/operator/scalar/sql/ArraySqlFunctions.java index b51855cc9a2ee..2fc1218ea2408 100644 --- a/presto-main/src/main/java/com/facebook/presto/operator/scalar/sql/ArraySqlFunctions.java +++ b/presto-main/src/main/java/com/facebook/presto/operator/scalar/sql/ArraySqlFunctions.java @@ -69,7 +69,7 @@ public static String arrayFrequency() "m -> m)"; } - @SqlInvokedScalarFunction(value = "array_duplicates", alias = {"array_dupes"}, deterministic = true, calledOnNullInput = false) + @SqlInvokedScalarFunction(value = "array_duplicates", deterministic = true, calledOnNullInput = false) @Description("Returns set of elements that have duplicates") @SqlParameter(name = "input", type = "array(T)") @TypeParameter("T") @@ -81,7 +81,7 @@ public static String arrayDuplicates() "map_keys(map_filter(array_frequency(input), (k, v) -> v > 1)))"; } - @SqlInvokedScalarFunction(value = "array_has_duplicates", alias = {"array_has_dupes"}, deterministic = true, calledOnNullInput = false) + @SqlInvokedScalarFunction(value = "array_has_duplicates", deterministic = true, calledOnNullInput = false) @Description("Returns whether array has any duplicate element") @TypeParameter("T") @SqlParameter(name = "input", type = "array(T)") diff --git a/presto-main/src/test/java/com/facebook/presto/operator/scalar/sql/TestArraySqlFunctions.java b/presto-main/src/test/java/com/facebook/presto/operator/scalar/sql/TestArraySqlFunctions.java index bcc96309d96ad..2e0e865f13be4 100644 --- a/presto-main/src/test/java/com/facebook/presto/operator/scalar/sql/TestArraySqlFunctions.java +++ b/presto-main/src/test/java/com/facebook/presto/operator/scalar/sql/TestArraySqlFunctions.java @@ -196,9 +196,6 @@ public void testArrayHasDuplicates() assertFunction("array_has_duplicates(array[0, null])", BOOLEAN, false); assertFunction("array_has_duplicates(array[0, null, null])", BOOLEAN, true); - // Test legacy name. - assertFunction("array_has_dupes(array[varchar 'a', varchar 'b', varchar 'a'])", BOOLEAN, true); - assertFunction("array_has_duplicates(array[array[1], array[2], array[]])", BOOLEAN, false); assertFunction("array_has_duplicates(array[array[1], array[2], array[2]])", BOOLEAN, true); assertFunction("array_has_duplicates(array[(1, 2), (1, 2)])", BOOLEAN, true); @@ -224,9 +221,6 @@ public void testArrayDuplicates() assertFunction("array_duplicates(array[0, null])", new ArrayType(INTEGER), ImmutableList.of()); assertFunction("array_duplicates(array[0, null, null])", new ArrayType(INTEGER), singletonList(null)); - // Test legacy name. - assertFunction("array_dupes(array[1, 2, 1])", new ArrayType(INTEGER), ImmutableList.of(1)); - RowType rowType = RowType.from(ImmutableList.of(RowType.field(INTEGER), RowType.field(INTEGER))); assertFunction("array_duplicates(array[array[1], array[2], array[]])", new ArrayType(new ArrayType(INTEGER)), ImmutableList.of()); assertFunction("array_duplicates(array[array[1], array[2], array[2]])", new ArrayType(new ArrayType(INTEGER)), ImmutableList.of(ImmutableList.of(2)));