You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are still a few gaps in DefaultVectorComparators:
FixedSizeBinaryVector
LargeListVector
FixedSizeListVector
NullVector is also missing one, though sorting is irrelevant. Perhaps a trivial comparator that just returns equality always.
StructVector does not really make sense to make comparable.
Component(s)
Java
The text was updated successfully, but these errors were encountered:
### Rationale for this change
Add missing Default VectorValueComparators for some more types.
### What changes are included in this PR?
Add comparators for:
- FixedSizeBinaryVector
- LargeListVector
- FixedSizeListVector
- NullVector
### Are these changes tested?
Yes, unit tests added.
### Are there any user-facing changes?
No
* Closes: #38662
Authored-by: James Duong <james.duong@improving.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
dgreiss
pushed a commit
to dgreiss/arrow
that referenced
this issue
Feb 19, 2024
### Rationale for this change
Add missing Default VectorValueComparators for some more types.
### What changes are included in this PR?
Add comparators for:
- FixedSizeBinaryVector
- LargeListVector
- FixedSizeListVector
- NullVector
### Are these changes tested?
Yes, unit tests added.
### Are there any user-facing changes?
No
* Closes: apache#38662
Authored-by: James Duong <james.duong@improving.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Describe the enhancement requested
There are still a few gaps in DefaultVectorComparators:
NullVector is also missing one, though sorting is irrelevant. Perhaps a trivial comparator that just returns equality always.
StructVector does not really make sense to make comparable.
Component(s)
Java
The text was updated successfully, but these errors were encountered: