dlmarion commented on code in PR #5532:
URL: https://github.com/apache/accumulo/pull/5532#discussion_r2076013466
##########
core/src/main/java/org/apache/accumulo/core/spi/compaction/CompactionPlanner.java:
##########
@@ -94,6 +95,12 @@ public interface PlanningParameters {
*/
TableId getTableId();
+ /**
+ * @return the tablet for which a compaction is being planned
+ * @since 2.1.4
+ */
+ TabletId getTabletId();
Review Comment:
I think for a 3.1 and a 4.0 we could remove the default method that returns
null and just leave the method declaration in the interface. I think this
change (and the other that you pointed out) potentially make a 2.1.4 api
implementation not backwards compatible.
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
