kevinrr888 commented on code in PR #5031:
URL: https://github.com/apache/accumulo/pull/5031#discussion_r2071638661
##########
core/src/main/java/org/apache/accumulo/core/fate/user/UserFateStore.java:
##########
@@ -270,9 +271,10 @@ protected Stream<FateIdStatus>
getTransactions(EnumSet<TStatus> statuses) {
Scanner scanner = context.createScanner(tableName, Authorizations.EMPTY);
scanner.setRange(new Range());
RowFateStatusFilter.configureScanner(scanner, statuses);
- TxColumnFamily.STATUS_COLUMN.fetch(scanner);
- TxColumnFamily.RESERVATION_COLUMN.fetch(scanner);
- TxInfoColumnFamily.FATE_OP_COLUMN.fetch(scanner);
+ // columns fetched from TxAdminColumnFamily
Review Comment:
```suggestion
// columns fetched here must be in/added to TxAdminColumnFamily for
locality group benefits
```
My point with the comment was to ensure if we fetch more columns here in the
future that we move them to TxAdmin for the locality group benefits.
--
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
