From d8d50bd11e0bc8b35b6724e56d17ea4479fe49e1 Mon Sep 17 00:00:00 2001 From: Xiao Du Date: Tue, 25 Feb 2025 20:09:11 -0800 Subject: [PATCH] test(fuzzer): Skip map_keys_by_top_n_values Test (#12447) Summary: The fuzzer test for map_keys_by_top_n_values should be skipped until the presto array comparison null catch patch is fully released to the public. The only failure scenario occurs when the input value is an array with a null element, which is low risk and has already been fixed in this https://github.com/prestodb/presto/pull/24570. Once the patch is fully rolled out, we can remove the method from the skip list. Reviewed By: kgpai, yuandagits, HeidiHan0000 Differential Revision: D70198160 --- velox/expression/fuzzer/ExpressionFuzzerTest.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/velox/expression/fuzzer/ExpressionFuzzerTest.cpp b/velox/expression/fuzzer/ExpressionFuzzerTest.cpp index 914862602449..73e19bebce5b 100644 --- a/velox/expression/fuzzer/ExpressionFuzzerTest.cpp +++ b/velox/expression/fuzzer/ExpressionFuzzerTest.cpp @@ -198,6 +198,8 @@ int main(int argc, char** argv) { "current_date", // Non-deterministic "xxhash64_internal", "combine_hash_internal", + "map_keys_by_top_n_values", // requires + // https://github.com/prestodb/presto/pull/24570 }); referenceQueryRunner = std::make_shared(