EdColeman commented on issue #3892: URL: https://github.com/apache/accumulo/issues/3892#issuecomment-1779543767
It is an interface and that is just providing a default implementation. Is it not the responsibility of the classes that extend it to provide an implementation? The fact that it throws an UnsupportedOperationException is true. The fact that it will be removed in a future version is also true. If the message was "this operation is not implemented and must be overridden" would be accurate, but sort of redundant with the UnsupportedOperationException itself. Stating that it will be removed may be help for developers - rather than just implementing the method, maybe they'd choose another path? The message may not be the most user friendly message, but it is providing accurate information and does not seem to be a bug. The code is functionally correct - unless you think there should be a different default implementation? -- 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]
