kfaraz commented on PR #17782: URL: https://github.com/apache/druid/pull/17782#issuecomment-2744965596
@vogievetsky , had you updated the config in a prior API call to use `engine: msq`? You can verify that by doing a `GET /druid/coordinator/v1/config/compaction`. (On that note, I think we should also add a separate `GET /druid/coordinator/v1/config/compaction/cluster`). The update API `POST /druid/coordinator/v1/config/compaction/cluster` only updates the configs that have been specified in the payload and leaves the rest unchanged. This was done to align with the API `POST /druid/coordinator/v1/config` (i.e. update coordinator dynamic config) which behaves in a similar way. I would personally prefer if both of these APIs (or atleast the new one) just updated the whole object (since that is what most users would expect anyway). Please let me know what you think. I will make the necessary changes. -- 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]
