Subham-KRLX commented on PR #3960:
URL: https://github.com/apache/polaris/pull/3960#issuecomment-4026049160

   @dimas-b thanks for the review I have streamlined the implementation based 
on your feedback by refactoring StoreType into an enum within 
DataSourceResolver and removing the getAllUniqueDataSources() method to keep 
the interface focused strictly on runtime resolution. Additionally I moved 
DefaultDataSourceResolver
    to the runtime-common module to better decouple the persistence layer from 
resolution logic and simplified the implementation by injecting a plain 
DataSource instead of an Instance.


-- 
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]

Reply via email to