keith-turner commented on code in PR #3232:
URL: https://github.com/apache/accumulo/pull/3232#discussion_r1144068116


##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/MetadataSchema.java:
##########
@@ -226,6 +226,12 @@ public static void validateDirCol(String dirName) {
        */
       public static final String LOCK_QUAL = "lock";
       public static final ColumnFQ LOCK_COLUMN = new ColumnFQ(NAME, new 
Text(LOCK_QUAL));
+
+      public static final String OP_QUAL = "op";
+      public static final ColumnFQ OP_COLUMN = new ColumnFQ(NAME, new 
Text(OP_QUAL));
+
+      public static final String OPID_QUAL = "opid";
+      public static final ColumnFQ OPID_COLUMN = new ColumnFQ(NAME, new 
Text(OPID_QUAL));

Review Comment:
   I think its ok here.  In the past tablet col fam was meant to hold info 
needed by clients and had its own column family.  However I am not sure how 
relevant that it.  I think we need to reevaluate the metadata schema and 
locality groups against the different scan patterns.  Also should probably  
look into passing hints for certain metadata scans that cause them to not use 
cache or use cache.  Should probably do that after the dust settles on the 
elasticity changes.



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

Reply via email to