gszadovszky commented on code in PR #3597:
URL: https://github.com/apache/parquet-java/pull/3597#discussion_r3499996314
##########
parquet-column/src/main/java/org/apache/parquet/filter2/columnindex/RowRanges.java:
##########
@@ -316,4 +316,72 @@ public List<Range> getRanges() {
public String toString() {
return ranges.toString();
}
+
+ /**
Review Comment:
Oh, I haven't noticed that, sorry. Then, we probably need to get back to the
deprecated/move pattern. :disappointed:
Let's move to the new place, extend the class as required at the old place
and deprecate. Also deprecate the public API that references the old place and
create new methods for the new one. Mark 2.0 as the removal version. WDYT?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
