Skip to content

Commit

Permalink
Extract AggregationFuzzerBase from AggregationFuzzer (#7916)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #7916

Extract AggregationFuzzerBase from AggregationFuzzer. This is needed for building WindowFuzzer
that reuses common logic in AggregaitonFuzzerBase.

This diff also moves fuzzer-related files to
velox/exec/fuzzer, including
– AggregationFuzzer.h/cpp
– AggregationFuzzerRunner.h
– ReferenceQueryRunner.h
– DuckQueryRunner.h/cpp
– PrestoQueryRunner.h/cpp
– InputGenerator.h
– ResultVerifier.h
– TransformResultVerifier.h

It also moves the Presto functions' custom input generators and result verifiers to
velox/functions/prestosql/fuzzer, including
– MinMaxInputGenerator.h
– ApproxDistinctInputGenerator.h
– ApproxDistinctResultVerifier.h
– ApproxPercentileInputGenerator.h
– ApproxPercentileResultVerifier.h

This is the first piece of #7754.

Reviewed By: mbasmanova

Differential Revision: D51692940

fbshipit-source-id: cf8aec101f9d0b8d47a0db1e66e5f9b651f10414
  • Loading branch information
kagamiori authored and facebook-github-bot committed Dec 14, 2023
1 parent 2ad0686 commit 87cb8c1
Show file tree
Hide file tree
Showing 28 changed files with 2,183 additions and 1,710 deletions.
1 change: 1 addition & 0 deletions velox/exec/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,7 @@ target_link_libraries(
velox_common_compression)

if(${VELOX_BUILD_TESTING})
add_subdirectory(fuzzer)
add_subdirectory(tests)
elseif(${VELOX_BUILD_TEST_UTILS})
add_subdirectory(tests/utils)
Expand Down
Loading

0 comments on commit 87cb8c1

Please sign in to comment.