dlmarion commented on code in PR #5498:
URL: https://github.com/apache/accumulo/pull/5498#discussion_r2056729968
##########
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 prefer to call them the system namespace / system tables, but that may be
due to my experience with relational databases.. Having Table and Namespace
objects for the public API might make sense at some point. Having an internal
object that represents the objects necessary to maintain the system also makes
sense, I think we just need the right naming.
By the way, we also have an object called `RootTable` which mostly contains
constants that could be in `Constants.java`.
--
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
