frankjkelly edited a comment on issue #7893: URL: https://github.com/apache/pulsar/issues/7893#issuecomment-682582539
Thanks @sijie thanks for the clarification. I guess I don't get the rationale for implementing a default method that just throws an exception rather than an abstract method which tells the developer "Hey you now have to implement these methods"? For example https://github.com/apache/pulsar/blob/48f5a2f62c148b3df617be060fefed51f3145979/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java#L262-L269 or https://github.com/apache/pulsar/blob/48f5a2f62c148b3df617be060fefed51f3145979/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java#L434-L440 Fair enough if there is a reason but also is there someway to improve the exception/stack trace logging in standalone. So I don't just see ```19:40:12.304 [main] INFO org.apache.pulsar.PulsarStandalone - HTTP 500 Internal Server Error``` If you point me to the right part of the code I'd be happy to throw together a PR. ---------------------------------------------------------------- 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]
