suneet-s commented on a change in pull request #11164:
URL: https://github.com/apache/druid/pull/11164#discussion_r625156598



##########
File path: docs/configuration/index.md
##########
@@ -750,6 +750,9 @@ These Coordinator static configurations can be defined in 
the `coordinator/runti
 |`druid.coordinator.kill.audit.on`| Boolean value for whether to enable 
automatic deletion of audit logs. If set to true, Coordinator will periodically 
remove audit logs from the audit table entries in metadata storage.| No | 
False| 
 |`druid.coordinator.kill.audit.period`| How often to do automatic deletion of 
audit logs in [ISO 8601](https://en.wikipedia.org/wiki/ISO_8601) duration 
format. Value must be greater than 
`druid.coordinator.period.metadataStoreManagementPeriod`. Only applies if 
`druid.coordinator.kill.audit.on` is set to True.| No| `P1D`|
 |`druid.coordinator.kill.audit.durationToRetain`| Duration of audit logs to be 
retained from created time in [ISO 
8601](https://en.wikipedia.org/wiki/ISO_8601) duration format. Only applies if 
`druid.coordinator.kill.audit.on` is set to True.| Yes if 
`druid.coordinator.kill.audit.on` is set to True| None|
+|`druid.coordinator.kill.rule.on`| Boolean value for whether to enable 
automatic deletion of rules. If set to true, Coordinator will periodically 
remove rules of inactive datasource (datasource with no used and unused 
segments) from the rule table in metadata storage.| No | False| 
+|`druid.coordinator.kill.rule.period`| How often to do automatic deletion of 
rules in [ISO 8601](https://en.wikipedia.org/wiki/ISO_8601) duration format. 
Value must be greater than 
`druid.coordinator.period.metadataStoreManagementPeriod`. Only applies if 
`druid.coordinator.kill.rule.on` is set to True.| No| `P1D`|
+|`druid.coordinator.kill.rule.durationToRetain`| Duration of rules to be 
retained from created time in [ISO 
8601](https://en.wikipedia.org/wiki/ISO_8601) duration format. Only applies if 
`druid.coordinator.kill.rule.on` is set to True.| Yes if 
`druid.coordinator.kill.rule.on` is set to True| None|

Review comment:
       We should update this to say that this will only work as expected if the 
version is a timestamp




-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to