dlmarion commented on code in PR #5498:
URL: https://github.com/apache/accumulo/pull/5498#discussion_r2056634562
##########
core/src/main/java/org/apache/accumulo/core/metadata/AccumuloNamespace.java:
##########
@@ -28,7 +28,7 @@
/**
* Defines the name and id of all tables in the accumulo table namespace.
*/
-public enum AccumuloTable {
+public enum AccumuloNamespace {
Review Comment:
I'm not sure that this is the best approach. If you look at
`org.apache.accumulo.core.clientImpl.Namespace` I think you will find that
`AccumuloTable` could become `Table.java` and that AccumuloTable.java is just a
specialization of that to represent things in the Accumulo namespace. We have
`Namespace.java` as a first class citizen, but there is no `Table.java`.
`AccumuloTable` is more of a utility class than anything. I'm not sure what the
right answer is here, but I'm not sure this is it.
--
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
