vibhatha commented on code in PR #43077: URL: https://github.com/apache/arrow/pull/43077#discussion_r1692357593
########## 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: Right, what I meant was the content for default here? I already removed this part from this PR for a followup PR. As we can do visitors in a separate PR. For the default implementation would it be okay to throw Unsupported as the default behavior? -- 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