Skip to content

Commit

Permalink
Move JoinFuzzer into exec/fuzzer (facebookincubator#9788)
Browse files Browse the repository at this point in the history
Summary:
Currently,  `JoinFuzzer` related files are mixed with test files under exec/tests.
Move JoinFuzzer into exec/fuzzer to align with other Fuzzers.

Pull Request resolved: facebookincubator#9788

Reviewed By: tanjialiang

Differential Revision: D57314361

Pulled By: xiaoxmeng

fbshipit-source-id: 1ee7889b847dfcd554dce6c9624609f928a67d4c
  • Loading branch information
yanngyoung authored and facebook-github-bot committed May 14, 2024
1 parent b29d933 commit 43cd80a
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 9 deletions.
5 changes: 5 additions & 0 deletions velox/exec/fuzzer/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -63,3 +63,8 @@ add_library(velox_row_number_fuzzer RowNumberFuzzer.cpp)
target_link_libraries(
velox_row_number_fuzzer velox_fuzzer_util velox_type velox_vector_fuzzer
velox_exec_test_lib velox_expression_test_utility)

add_library(velox_join_fuzzer JoinFuzzer.cpp)

target_link_libraries(velox_join_fuzzer velox_type velox_vector_fuzzer
velox_exec_test_lib velox_expression_test_utility)
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#include "velox/exec/tests/JoinFuzzer.h"
#include "velox/exec/fuzzer/JoinFuzzer.h"
#include <boost/random/uniform_int_distribution.hpp>
#include "velox/common/file/FileSystems.h"
#include "velox/connectors/hive/HiveConnector.h"
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
#include "velox/common/file/FileSystems.h"
#include "velox/common/memory/SharedArbitrator.h"
#include "velox/exec/MemoryReclaimer.h"
#include "velox/exec/tests/JoinFuzzer.h"
#include "velox/exec/fuzzer/JoinFuzzer.h"
#include "velox/serializers/PrestoSerializer.h"

/// Join FuzzerRunner leverages JoinFuzzer and VectorFuzzer to
Expand Down
7 changes: 1 addition & 6 deletions velox/exec/tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -201,18 +201,13 @@ target_link_libraries(
velox_tpch_connector
velox_memory)

# Join Fuzzer.
add_library(velox_join_fuzzer JoinFuzzer.cpp)

target_link_libraries(velox_join_fuzzer velox_type velox_vector_fuzzer
velox_exec_test_lib velox_expression_test_utility)

# RowNumber Fuzzer.
add_executable(velox_row_number_fuzzer_test RowNumberFuzzerTest.cpp)

target_link_libraries(velox_row_number_fuzzer_test velox_row_number_fuzzer
gtest gtest_main)

# Join Fuzzer.
add_executable(velox_join_fuzzer_test JoinFuzzerTest.cpp)

target_link_libraries(velox_join_fuzzer_test velox_join_fuzzer gtest gtest_main)
Expand Down
2 changes: 1 addition & 1 deletion velox/exec/tests/JoinFuzzerTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
#include <gtest/gtest.h>
#include <unordered_set>

#include "velox/exec/tests/JoinFuzzerRunner.h"
#include "velox/exec/fuzzer/JoinFuzzerRunner.h"

DEFINE_int64(
seed,
Expand Down

0 comments on commit 43cd80a

Please sign in to comment.