hddong commented on pull request #908: URL: https://github.com/apache/incubator-kyuubi/pull/908#issuecomment-927264101
> we can do filtering based on the contents of the events, e.g. state? @yaooqinn ： Sorry for feedback, been busy with other things lately. `onEvent` log all events, but not all `KyuubiServerEvent` have `state`. We can reuse loggers, and add `auditLoggers` and `onAuditEvent` in `EventLoggingService` to make events and audit relatively independent. Then, we can choose to use one or all of them as needed. -- 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]
