Subham-KRLX commented on PR #3960: URL: https://github.com/apache/polaris/pull/3960#issuecomment-4045943100
> As I said in the mailing thread https://lists.apache.org/thread/nokyythvrdzsfwz26hx0w5rpxrw5wjtd, I think this effort deserve more thoughts before we start coding. cc @singhpk234 @flyrain I agree that the full metrics/events routing story needs a broader design discussion! To address this exact concern (raised here and by EJ Wang), I just pushed an update to significantly narrow the scope of this PR. I have entirely removed METRICS and EVENTS—this PR is now solely establishing the DataSourceResolver SPI foundation for the METASTORE. This gives downstream users a cleaner injection point without locking in any architecture for the new tables. Does this narrower first step alleviate your concerns for this specific PR? -- 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]
