vtlim commented on code in PR #13131: URL: https://github.com/apache/druid/pull/13131#discussion_r980535420
########## docs/operations/clean-metadata-store.md: ########## @@ -143,7 +143,7 @@ Datasource cleanup uses the following configuration: ### Indexer task logs -You can configure the Overlord to delete indexer task log metadata and the indexer task logs from local disk or from cloud storage. +You can configure the Overlord to delete indexer task log metadata and the indexer task logs from local disk or from cloud storage. The cleanup includes the `druid_tasks` and `druid_tasklogs` tables in the metadata database, and the task logs in deep storage. (Note that `druid_tasklogs` is no longer used and will already be empty, unless the druid version is older.) Review Comment: Two follow up questions: 1. Do you know what older version uses `druid_tasklogs`? Wondering if we can say something like "unless the Druid version is XX or older." 2. Should the table be called `druid_taskLog`? That's what's listed in the configuration table: https://druid.apache.org/docs/latest/configuration/index.html#metadata-storage -- 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]
