kfaraz commented on code in PR #15039:
URL: https://github.com/apache/druid/pull/15039#discussion_r1356667308
##########
indexing-service/src/main/java/org/apache/druid/indexing/overlord/TaskStorageQueryAdapter.java:
##########
@@ -60,6 +61,15 @@ public List<Task> getActiveTasks()
return storage.getActiveTasks();
}
+ /**
+ * @param lockFilterPolicies Requests for conflicing lock intervals for
various datasources
+ * @return Map from datasource to intervals locked by tasks that have a
conflicting lock type that cannot be revoked
+ */
+ public Map<String, List<Interval>>
getLockedIntervalsV2(List<LockFilterPolicy> lockFilterPolicies)
Review Comment:
Might make more sense to have a per datasource method here instead. The
method in OverlordResource itself can build the map.
--
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]
