ihuzenko commented on a change in pull request #1831: DRILL-7331: Drill Iceberg
Metastore metadata expiration
URL: https://github.com/apache/drill/pull/1831#discussion_r309167366
##########
File path:
metastore/iceberg-metastore/src/main/java/org/apache/drill/metastore/iceberg/MetastoreContext.java
##########
@@ -50,4 +51,11 @@
* @return transformer instance
*/
Transformer<T> transformer();
+
+ /**
+ * Returns expiration handler that expires Iceberg table metadata.
+ *
+ * @return expiration handler instance
+ */
+ ExpirationHandler expirationHandler();
Review comment:
This ```MetastoreContext``` interface consists of getters without get
prefix... Looks a little bit strange for me. But I'm ok with it if it does not
violate the Drill's conventions.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
