deniskuzZ commented on code in PR #4091: URL: https://github.com/apache/hive/pull/4091#discussion_r1161672912
########## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnStore.java: ########## @@ -516,6 +516,19 @@ Set<CompactionInfo> findPotentialCompactions(int abortedThreshold, long abortedT @RetrySemantics.ReadOnly List<CompactionInfo> findReadyToClean(long minOpenTxnWaterMark, long retentionTime) throws MetaException; + /** + * Find the aborted entries in TXN_COMPONENTS which can be used to + * clean directories belonging to transactions in aborted state. + * @param abortedTimeThreshold Age of table/partition's oldest aborted transaction involving a given table + * or partition that will trigger cleanup. + * @param abortedThreshold Number of aborted transactions involving a given table or partition + * that will trigger cleanup. + * @return Information of potential abort items that needs to be cleaned. + * @throws MetaException + */ + @RetrySemantics.ReadOnly + List<AcidTxnInfo> findReadyToCleanForAborts(long abortedTimeThreshold, int abortedThreshold) throws MetaException; Review Comment: maybe `findReadyToCleanAborts` ? -- 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]
