[ARROW-6266] [Java] Resolve the ambiguous method overload in RangeEqualsVisitor

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22650
• Assignee: Liya Fan
• Reporter: Liya Fan
• Created: 16/Aug/19 06:36
• Updated: 11/Jan/23 07:46
• Resolved: 19/Aug/19 12:04

Description

In RangeEqualsVisitor, there are overload methods for both super class and sub class. This will lead to unexpected behavior.

For example, if we call RangeEqualsVisitor#visit(v), where v is a fixed width vector, the method actually called may be visit(ValueVector), which is unexpected.

In general, in the visitor pattern, it is not a good idea to support method overload for both super class and sub-class as parameters.

Issue Links

• blocks: ARROW-6113 [Java] Support vector deduplicate function
• links to: GitHub Pull Request #5100