ddcprg commented on a change in pull request #7541:
URL: https://github.com/apache/pinot/pull/7541#discussion_r741708400
##########
File path:
pinot-common/src/main/java/org/apache/pinot/common/metadata/segment/SegmentZKMetadata.java
##########
@@ -410,21 +408,4 @@ public TimeUnit getTimeUnit() {
return null;
}
}
-
Review comment:
This code is deprecated and it doesn't look like is used anywhere in
Pinot - I've checked for the string version of the method names in case
reflection was used. I don't think projects other than Pinot should be using
this class, I've searched in both Trino and Presto and the class doesn't seem
to be in use
--
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]
