Amemeda commented on PR #6456: URL: https://github.com/apache/accumulo/pull/6456#issuecomment-4856493611
> 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. would the resy of the files in minicluster and miniclusterImpl have to move to the new package as well? <img width="475" height="485" alt="image" src="https://github.com/user-attachments/assets/67d19e09-7f0d-4730-8022-b6dc76736f6a"; /> -- 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]
