-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
GH-35316: [C++][FlightSQL] Use RowsToBatches() instead of ArrayFromJSON() in SQLite example server #35322
Conversation
…FromJSON() in SQLite example server Because ArrayFromJSON() requires arrow_testing. But cpp/examples/arrow/flight_sql_example.cc uses the SQLite example server without arrow_testing dependency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This is a bit complex though for what I would think is a common case. Perhaps we should consider some better utilities for this. (ADBC uses a column-wise, templated helper, though I suppose column-wise would also be messy here. https://github.com/apache/arrow-adbc/blob/a1dd00f88d76beaccfce000098d4cdb3944c015d/c/validation/adbc_validation_util.h#L304-L311)
I agree with you. |
Benchmark runs are scheduled for baseline = 2444253 and contender = 797f346. 797f346 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
…FromJSON() in SQLite example server (apache#35322) ### Rationale for this change `ArrayFromJSON()` requires `libarrow_testing.so`. But `cpp/examples/arrow/flight_sql_example.cc` uses the SQLite example server without `libarrow_testing.so` dependency. ### What changes are included in this PR? Use `RowsToBatches()` in `libarrow.so` instead of `ArrayFromJSON()`. ### Are these changes tested? Yes by hand. ### Are there any user-facing changes? Yes. * Closes: apache#35316 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…FromJSON() in SQLite example server (apache#35322) ### Rationale for this change `ArrayFromJSON()` requires `libarrow_testing.so`. But `cpp/examples/arrow/flight_sql_example.cc` uses the SQLite example server without `libarrow_testing.so` dependency. ### What changes are included in this PR? Use `RowsToBatches()` in `libarrow.so` instead of `ArrayFromJSON()`. ### Are these changes tested? Yes by hand. ### Are there any user-facing changes? Yes. * Closes: apache#35316 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…FromJSON() in SQLite example server (apache#35322) ### Rationale for this change `ArrayFromJSON()` requires `libarrow_testing.so`. But `cpp/examples/arrow/flight_sql_example.cc` uses the SQLite example server without `libarrow_testing.so` dependency. ### What changes are included in this PR? Use `RowsToBatches()` in `libarrow.so` instead of `ArrayFromJSON()`. ### Are these changes tested? Yes by hand. ### Are there any user-facing changes? Yes. * Closes: apache#35316 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Rationale for this change
ArrayFromJSON()
requireslibarrow_testing.so
. Butcpp/examples/arrow/flight_sql_example.cc
uses the SQLite example server withoutlibarrow_testing.so
dependency.What changes are included in this PR?
Use
RowsToBatches()
inlibarrow.so
instead ofArrayFromJSON()
.Are these changes tested?
Yes by hand.
Are there any user-facing changes?
Yes.