BryanCutler commented on a change in pull request #35393:
URL: https://github.com/apache/spark/pull/35393#discussion_r799820612
##########
File path:
sql/catalyst/src/main/java/org/apache/spark/sql/vectorized/ArrowColumnVector.java
##########
@@ -130,9 +130,16 @@ public ColumnarMap getMap(int rowId) {
@Override
public ArrowColumnVector getChild(int ordinal) { return
childColumns[ordinal]; }
+ ArrowColumnVector(DataType type) {
Review comment:
So is the idea to call this constructor and then override `initAccessor`
to provide a custom accessor?
If so, would it be easier to just add a constructor `public
ArrowColumnVector(ArrowVectorAccessor accessor)` with the relaxed access
modifiers?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
