jtuglu-netflix commented on PR #18082: URL: https://github.com/apache/druid/pull/18082#issuecomment-2963662077
> > Yes, mainly for reference. Although there are still some things which rely on a corresponding datasource, like DerivativeDataSourceManager.java. This file can be modified to get the datasource reference elsewhere, but I left as-is. > > Hmm, I see. I think we should be able to use `supervisor_id` for this one too since it will effectively be the same. But I will try to take a look at the relevant code to be sure. It will mean updates to the method [here](https://github.com/apache/druid/blob/-/extensions-contrib/materialized-view-selection/src/main/java/org/apache/druid/query/materializedview/DerivativeDataSourceManager.java#L186). -- 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]
