Details
-
Sub-task
-
Status: Resolved
-
Major
-
Resolution: Won't Fix
-
None
-
None
-
None
-
None
Description
AccumuloSecurityException has a public constructor which is completely unused. I don't know if we should consider constructors of exceptions "public API", but they are defined as such in the README, so a strict interpretation is that they are.
The unused constructor in question is:
public AccumuloSecurityException(final String user, final SecurityErrorCode errorcode, final String tableInfo) { ... }
This class also uses thrift objects in its constructors, which we don't consider public API, so either we should also consider constructors of exceptions to be not public API (in which case, we can just delete the unused constructor), or we should consider the use of thrift objects bugs.
If these constructors are public API, they should have tests and we shouldn't have any unused (because they would minimally be referenced in tests).