nandorKollar commented on PR #4803: URL: https://github.com/apache/polaris/pull/4803#issuecomment-4752319516
> @nandorKollar : just FYI: this class is used in Apache Polaris code only on "in memory" persistence call paths. > > So this fix in OSS is probably not very critical... especially considering https://lists.apache.org/thread/nzoljc1ohnsq4f5o28dh4opqkqw3p09h > > Do you need this fix for a downstream build? You were quicker than me. :) I wanted to mention the same after I submitted my comment, probably we should hold off on merging this, as as far as understood, the plan is to deprecate and remove this. No, afaik we don't need this fix downstream (@ayushtkn correct me if I'm wrong). -- 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]
