Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20277#discussion_r162876178
--- 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 --
It is good to do it later. I agree that we do the same check at one place.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
