dimas-b commented on PR #3699: URL: https://github.com/apache/polaris/pull/3699#issuecomment-4240346414
> [@dimas-b](https://github.com/dimas-b) I saw you had previously removed [#1480](https://github.com/apache/polaris/pull/1480) from Vendor but left it on the integration layer, any reason for that? from what I saw it's completely unused and can be deleted from there as well. [...] I did not touch `PolarisStorageIntegration.validateAccessToLocations()` because it is considered part of the historical "core" and I am now aware of the full implications of such a change. From my POV that method can be deleted, but I'd appreciate it if @dennishuo could share his view on that. -- 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]
