gianm commented on PR #14810: URL: https://github.com/apache/druid/pull/14810#issuecomment-1680833822
Architecturally it seems weirdly complicated for the `compact` task to start a new top-level controller task. It will also probably lead to issues with locking. One of these seems like a better approach to me. What do you think of these approaches? 1. Have the `compact` task itself be an MSQ controller task (similar to how it can itself be an `index_parallel` controller task). It would need to generate a native query rather than a SQL query for this to work, because it doesn't have direct access to the SQL planner. 2. Forget the `compact` task, focus on auto-compaction. Have the Coordinator submit SQL statements rather than `compact` tasks. -- 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]
