jtuglu-netflix commented on issue #18008: URL: https://github.com/apache/druid/issues/18008#issuecomment-2940800383
@kfaraz I'm slightly against supervisor UUIDs since that might make things less readable on the console (users would have to look at the spec to get an idea of what topic, etc. it's pulling from). My thought is we keep the same supervisor ID scheme and just validate as we normally do via the `id` uniqueness constraint in `druid_supervisors` table. I agree that `druid_dataSource` would need a `supervisor_id` column. As for APIs, I feel like keeping them the same for now is fine. The only new addition might be getting the full list of supervisors for a given datasource, which can be done by pulling the list of supervisors from `druid_supervisors` and filtering on datasource. -- 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]
