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


##########
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:
   System is a well-known class name in Java, and I think that something like 
that may be too generically named. SystemSchema might work, but I'm having 
trouble thinking of this as a "schema" in any sense of that word I'm familiar 
with. "SystemTables" seemed to satisfy what you were thinking to me. At its 
most terse, maybe:
   
   ```java
   Namespace namespace();
   Map<TableId,String> mapping();
   ```
   
   In use:
   ```java
   boolean isSystemNamespace = 
SystemTables.namespace().id().equals(namespaceId);
   boolean isSystemTable = SystemTables.mapping().keySet().contains(tableId);
   ```
   
   But that's a lot of boilerplate, so having extra methods might help:
   
   ```java
   String namespaceName();
   NamespaceId namespaceId();
   Set<TableId> tableIds();
   boolean containsTableId(TableId tableId);
   boolean containsTableName(String tableName);
   Map<String,String> tableIdToNameMap();
   ```
   
   Either way, terse or verbose, I think the name "SystemTables" works.



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