cshannon commented on code in PR #5335:
URL: https://github.com/apache/accumulo/pull/5335#discussion_r1956458410
##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java:
##########
@@ -202,7 +206,74 @@ public enum ColumnType {
COMPACTED,
USER_COMPACTION_REQUESTED,
UNSPLITTABLE,
- MERGEABILITY
+ MERGEABILITY;
+
+ public static Set<ByteSequence> resolveFamilies(Set<ColumnType> columns) {
Review Comment:
The only issue I see is that for fetch(), for some ColumnTypes we fetch
families and for some we fetch columns. So I think we may need to split the
columns into two groups to grab the type we need which should be doable. I
assume we don't want to load the entire family if we only care about one column
so we still need to keep this.
i'm working on changes for this now.
--
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]
