-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HBASE-24609 Move MetaTableAccessor out of hbase-client #1943
Conversation
Introduced a CatalogFamilyFormat to place the method for generating/parsing cells in catalog family. |
This is for simplifying the implementation for HBASE-24608. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
|
||
/** A regex for parsing server columns from meta. See above javadoc for meta layout */ | ||
private static final Pattern SERVER_COLUMN_PATTERN = | ||
Pattern.compile("^server(_[0-9a-fA-F]{4})?$"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replica_id is a integer? So it should only have [0-9]?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@InterfaceAudience.Private
String REPLICA_ID_FORMAT = "%04X";
I think it is in hex format?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@@ -193,8 +133,8 @@ | |||
* @param connection connection we're using | |||
* @param visitor Visitor invoked against each row in regions family. | |||
*/ | |||
public static void fullScanRegions(Connection connection, final Visitor visitor) | |||
throws IOException { | |||
public static void fullScanRegions(Connection connection, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this method be in ClientMTA altogether? Or you leave it here just to minimize code churn?
return p.add(CellBuilderFactory.create(CellBuilderType.SHALLOW_COPY).setRow(p.getRow()) | ||
.setFamily(HConstants.CATALOG_FAMILY) | ||
.setQualifier(CatalogFamilyFormat.getSeqNumColumn(replicaId)).setTimestamp(p.getTimestamp()) | ||
.setType(Type.Put).setValue(Bytes.toBytes(openSeqNum)).build()); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to figure the prinicipal to apply figuring when to put something in MTA and when to put it in CMTA? Thanks.
* </pre> | ||
*/ | ||
@InterfaceAudience.Private | ||
public class CatalogFamilyFormat { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like could be package private?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only for the 'info' CF?
No description provided.