joshelser commented on pull request #1210: URL: https://github.com/apache/phoenix/pull/1210#issuecomment-829460211
> Why the notion of a separate type of logger as opposed to increasing the scope of the queryLogger? The earlier commits on this review actually had them both in one class. It was a little complex to manage not getting a no-op instance when _either_ query logging or audit logging. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
