vibhatha commented on code in PR #43077:
URL: https://github.com/apache/arrow/pull/43077#discussion_r1689421245
##########
java/vector/src/main/java/org/apache/arrow/vector/compare/VectorVisitor.java:
##########
@@ -60,4 +61,6 @@ public interface VectorVisitor<OUT, IN> {
OUT visit(NullVector left, IN value);
OUT visit(ExtensionTypeVector<?> left, IN value);
+
+ OUT visit(ListViewVector left, IN value);
Review Comment:
Mmm, not sure if we can add a default implementation here. For the
`BaseVariableWidthViewVector` we did the same, we added the function and added
implementation. I assumed the breaking change would be okay.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
