Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

Fix the deprecated code warning in shuffle_split_test #229

Closed
JkSelf opened this issue Apr 6, 2021 · 0 comments · Fixed by #230
Closed

Fix the deprecated code warning in shuffle_split_test #229

JkSelf opened this issue Apr 6, 2021 · 0 comments · Fixed by #230
Labels
bug Something isn't working

Comments

@JkSelf
Copy link
Contributor

JkSelf commented Apr 6, 2021

Fix the following warning in shuffle_split_test.

/root/git/native-sql-engine/native-sql-engine/cpp/src/tests/shuffle_split_test.cc: In member function ‘arrow::Result<std::shared_ptr<arrow::RecordBatch> > sparkcolumnarplugin::shuffle::SplitterTest::TakeRows(const std::shared_ptr<arrow::RecordBatch>&, const string&)’: /root/git/native-sql-engine/native-sql-engine/cpp/src/tests/shuffle_split_test.cc:127:79: warning: ‘arrow::Result<std::shared_ptr<arrow::RecordBatch> > arrow::compute::Take(const arrow::RecordBatch&, const arrow::Array&, const arrow::compute::TakeOptions&, arrow::compute::ExecContext*)’ is deprecated: Deprecated in 1.0.0. Use Datum-based version [-Wdeprecated-declarations] 127 | arrow::compute::TakeOptions{}, &cntx); | ^ In file included from /usr/local/include/arrow/compute/api.h:29, from /root/git/native-sql-engine/native-sql-engine/cpp/src/tests/shuffle_split_test.cc:18: /usr/local/include/arrow/compute/api_vector.h:324:38: note: declared here 324 | Result<std::shared_ptr<RecordBatch>> Take( | ^~~~ /root/git/native-sql-engine/native-sql-engine/cpp/src/tests/shuffle_split_test.cc:127:79: warning: ‘arrow::Result<std::shared_ptr<arrow::RecordBatch> > arrow::compute::Take(const arrow::RecordBatch&, const arrow::Array&, const arrow::compute::TakeOptions&, arrow::compute::ExecContext*)’ is deprecated: Deprecated in 1.0.0. Use Datum-based version [-Wdeprecated-declarations] 127 | arrow::compute::TakeOptions{}, &cntx); | ^ In file included from /usr/local/include/arrow/compute/api.h:29, from /root/git/native-sql-engine/native-sql-engine/cpp/src/tests/shuffle_split_test.cc:18: /usr/local/include/arrow/compute/api_vector.h:324:38: note: declared here 324 | Result<std::shared_ptr<RecordBatch>> Take( | ^~~~

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants