You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/05/27 21:51:18 UTC

[GitHub] [arrow] BryanCutler commented on a change in pull request #6425: ARROW-6111: [Java] Support LargeVarChar and LargeBinary types

BryanCutler commented on a change in pull request #6425:
URL: https://github.com/apache/arrow/pull/6425#discussion_r431461407



##########
File path: java/vector/src/main/java/org/apache/arrow/vector/validate/ValidateVectorVisitor.java
##########
@@ -90,6 +91,11 @@ public Void visit(BaseVariableWidthVector vector, Void value) {
     return null;
   }
 
+  @Override
+  public Void visit(BaseLargeVariableWidthVector left, Void value) {
+    return null;

Review comment:
       is this another TODO? Would you mind making JIRAs for this and the other TODOs here so they can be tracked?




----------------------------------------------------------------
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.

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