Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(test): refactor bulk load function test #1616

Merged
merged 4 commits into from
Sep 21, 2023

Conversation

acelyc111
Copy link
Member

@acelyc111 acelyc111 commented Sep 20, 2023

#887

There is no functional changes, but only refactor the bulk_load function test.

@acelyc111 acelyc111 force-pushed the refactor_bulkload_test branch 2 times, most recently from 9957952 to 23ec5bf Compare September 20, 2023 11:20
@acelyc111 acelyc111 force-pushed the refactor_bulkload_test branch from 23ec5bf to 0d0a8bd Compare September 20, 2023 11:29
@acelyc111 acelyc111 force-pushed the refactor_bulkload_test branch from 5345623 to c6ec862 Compare September 20, 2023 13:56
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
src/test/function_test/bulk_load/test_bulk_load.cpp Outdated Show resolved Hide resolved
@acelyc111 acelyc111 requested a review from empiredan September 21, 2023 01:07
@empiredan empiredan merged commit 42136fa into apache:master Sep 21, 2023
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants