HuaHuaY commented on code in PR #50976:
URL: https://github.com/apache/arrow/pull/50976#discussion_r3860376754
##########
cpp/src/arrow/compute/kernels/scalar_cast_nested.cc:
##########
@@ -17,6 +17,7 @@
// Implementation of casting to (or between) list types
+#include <iostream>
Review Comment:
Do we need this header file?
##########
cpp/src/arrow/compute/kernels/scalar_cast_test.cc:
##########
@@ -3646,6 +3646,72 @@ TEST(Cast, ListToListOptionsPassthru) {
}
}
+TEST(Cast, ListViewToList) {
Review Comment:
In the issue https://github.com/apache/arrow/issues/50994, you said that
> which throws an unsupported type-casting error in compute kernels.
However, I can't see the relevant changes in your PR. I reverted the changes
other than the tests, ran the newly added tests, and there were no exceptions
which are reported in the issue.
Could you explain how the error in issue description is triggered? Or does
the issue description need to be revised?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
