adnanhemani commented on PR #1904: URL: https://github.com/apache/polaris/pull/1904#issuecomment-3169520983
> The current events aren't all in one file, so while I think grouping them that way is fine we should be consistent. Agreed, I can do this in the next round of revisions. But am waiting for the followup meeting regarding events to put further work in on this PR. > IIUC associated of different events is possible (and intended) on the EventListener side. This is correct as well. @snazy - does that resolve your concerns? -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
