Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162862761
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
---
@@ -33,18 +33,6 @@
private final ArrowVectorAccessor accessor;
private ArrowColumnVector[] childColumns;
- private void ensureAccessible(int index) {
- ensureAccessible(index, 1);
- }
-
- private void ensureAccessible(int index, int count) {
--- End diff --
How about we do it later? We need to find a central place to put this
check, instead of doing it in every implementation.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
