olsoloviov commented on PR #3293: URL: https://github.com/apache/polaris/pull/3293#issuecomment-3739252923
> A minor improvement that comes to my mind right now: when there is no listener (e.g. if `polaris.event-listener.type=no-op`), or when the listener is not "interested" in all events (e.g. the CloudWatch listener), instantiating a `PolarisEvent` + `PolarisEventMetadata` + `AttributeMap` is useless. We should come up with a way to conditionally create events only when necessary. Thanks for the suggestion, @adutra, I think it's be quite reasonable. Unless you prefer it to be fixed in this PR, I think I can do it in a follow up PR, since it is somewhat orthogonal to the current improvement. -- 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]
