adnanhemani commented on PR #2962: URL: https://github.com/apache/polaris/pull/2962#issuecomment-3630064718
As per #3217, I'm satisfied that there is a way forward for the code that would be introduced here. @vchag - can you please look into addressing the review comments and we can move forward with merging this through! To recap, we should make all events not emitting by default. And then we can review (in new PRs) handful of events at a time to ensure whether a redaction mixin is required and if the current redaction mixins are sufficient for each new event we are emitting. -- 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]
