sadikovi commented on code in PR #36427:
URL: https://github.com/apache/spark/pull/36427#discussion_r869759054


##########
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/WritableColumnVector.java:
##########
@@ -816,8 +816,8 @@ protected boolean isArray() {
    * Sets up the common state and also handles creating the child columns if 
this is a nested
    * type.
    */
-  protected WritableColumnVector(int capacity, DataType type) {
-    super(type);
+  protected WritableColumnVector(int capacity, DataType dataType) {

Review Comment:
   Oh, I had to change the variable name because it clashes with `type` 
property in the class. I can keep the same name and instead use `this.type` in 
the code below if you like.



-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to