ctubbsii commented on PR #6456: URL: https://github.com/apache/accumulo/pull/6456#issuecomment-4848614319
The weird thing about these ones is that these classes are supposed to represent a generic cluster, not specifically a minicluster (which is one specific instance of a cluster). So, I'm not sure it makes sense to move them into the minicluster package (which is listed as public API). If possible, it might make sense to move them into the server base jar instead of the minicluster jar, with a package name like org.apache.accumulo.server.cluster, that is not public API. -- 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]
