[Java] Remove dependency on RangeEqualsVisitor from ValueVector interface

[asfimport]

This is a follow-up from #4933

 

public interface VectorVisitor<OUT, IN, EX extends Exception> {..}

 

In ValueVector : 

public <OUT, IN, EX extends Exception> OUT accept(VectorVisitor<OUT, IN, EX> visitor, IN value) throws EX;

 

Reporter: Pindikura Ravindra / @pravindra
Assignee: Ji Liu / @tianchen92

Related issues:

• [Java] Implement ApproxEqualsVisitor comparing approx for floating point (blocks)
• [Java] Implement TypeEqualsVisitor comparing vector type equals considering names and metadata (blocks)

PRs and other links:

• GitHub Pull Request #5091

_{Note: This issue was originally created as ARROW-6211. Please see the migration documentation for further details.}

[asfimport]

Bryan Cutler / @BryanCutler:
So will this allow for other types of visitors besides a RangeEqualsVisitor?

[asfimport]

Pindikura Ravindra / @pravindra:
yes, I think it should.

[asfimport]

Ji Liu / @tianchen92:
@pravindra  One more question, 'IN value' above is unnecessary?

[asfimport]

Pindikura Ravindra / @pravindra:
It'll be useful for generic visitors to carry state (the IN value allows that). For the RangeEqualsVisitors, you can always use Void.

[asfimport]

Bryan Cutler / @BryanCutler:
This sounds good to me then, I agree it would be useful to have a generic visitor api

[asfimport]

Pindikura Ravindra / @pravindra:
Issue resolved by pull request 5091
#5091