JingsongLi commented on code in PR #9953:
URL: https://github.com/apache/paimon/pull/9953#discussion_r4051804116
##########
paimon-core/src/main/java/org/apache/paimon/table/source/AbstractDataEvolutionVectorRead.java:
##########
@@ -217,15 +222,25 @@ protected List<RoaringNavigableMap64>
preFilters(List<IndexVectorSearchSplit> sp
* support). {@code null} means "cannot decide", never "no rows match".
*/
@Nullable
+ /**
Review Comment:
This new javadoc was inserted after the old one, so `scalarMatchedRows` now
carries two stacked javadoc blocks (`@Nullable` is sandwiched between them at
line 224). The first block (lines 219-223) is the stale description of the
previous no-refinement behavior — could you drop it so only the new doc remains
directly above the method?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
