cecemei commented on code in PR #18950:
URL: https://github.com/apache/druid/pull/18950#discussion_r2755438483
##########
indexing-service/src/main/java/org/apache/druid/indexing/common/task/CompactionRunner.java:
##########
@@ -42,6 +42,28 @@ public interface CompactionRunner
{
String TYPE_PROPERTY = "type";
+ /**
+ * Returns whether this runner requires aligned intervals for compaction.
+ * When true, the compaction task will throw an error if the IOConfig has
allowNonAlignedInterval enabled.
+ *
+ * @return true if aligned intervals are required by this runner, false
otherwise. Default is true.
+ */
+ default boolean requireAlignedInterval()
+ {
+ return true;
+ }
+
+ /**
+ * Returns whether this runner requires dropExisting to be enabled for
compaction.
+ * When true, the compaction task will fail if dropExisting is not set to
true in the IOConfig.
+ *
+ * @return true if dropExisting must be enabled, false otherwise. Default is
true.
+ */
+ default boolean forceDropExisting()
+ {
+ return true;
Review Comment:
yea i made the dropExisting change in another pr:
https://github.com/apache/druid/pull/18968/changes#diff-280dab320bc1341105272f99d3ec8e7f8626c9d117f7affa933bc1a522bd465aL368,
but the check here is probably not all that necessary. anyway i removed this
check here so the task wont fail.
--
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]
