dlmarion commented on code in PR #5498:
URL: https://github.com/apache/accumulo/pull/5498#discussion_r2056805008


##########
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 was thinking one of:
   
     * a class called System, maybe in the org.apache.accumulo.core.schema 
package
     * or a call called SystemSchema
     
     the class would contain the system namespace and system tables and could 
have methods such as:
     
     `boolean isSystemNamespace(NamespaceId)`
     `NamespaceId getSystemNamespace()`
     `boolean isSystemTable(TableId)`
     `Set<TableId> getSystemTables()`



-- 
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

Reply via email to