DomGarguilo opened a new pull request, #5785: URL: https://github.com/apache/accumulo/pull/5785
This PR fixes the warnings from the core module. Here is a description from the ticket: > This warning gets emitted when an exception is thrown within an constructor. > > There are several ways to avoid this warning: > > 1. make the class final > 2. remove the thrown exception from the constructor > 3. add a `finalize()` method so it cant be overridden > 4. make the constructor private > > There are pros and cons to each fix. > > * making a class final prohibits inheriting it > * removing the thrown exception usually means forgoing validation > * adding a finalize() method means we also need to ignore the deprecation warning since finalizer() is deprecated > * making the constructor private usually takes a lot of refactoring to move to a different pattern to create the object > > I think the best approach here is to work through a couple modules at a time and fix these. -- 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: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
