dlmarion commented on code in PR #2665:
URL: https://github.com/apache/accumulo/pull/2665#discussion_r939103483
##########
server/base/src/main/java/org/apache/accumulo/server/metadata/ServerAmpleImpl.java:
##########
@@ -266,4 +273,71 @@ public Stream<ExternalCompactionFinalState>
getExternalCompactionFinalStates() {
throw new RuntimeException(e);
}
}
+
+ @Override
+ public void putScanServerFileReferences(Collection<ScanServerRefTabletFile>
scanRefs) {
+ try (BatchWriter writer =
context.createBatchWriter(DataLevel.USER.metaTable())) {
+ String prefix = ScanServerFileReferenceSection.getRowPrefix();
+ for (ScanServerRefTabletFile ref : scanRefs) {
+ Mutation m = new Mutation(prefix + ref.getRowSuffix());
+ m.put(ref.getServerAddress(), ref.getServerLockUUID(), ref.getValue());
+ writer.addMutation(m);
+ }
+ } catch (MutationsRejectedException | TableNotFoundException e) {
+ throw new RuntimeException(e);
Review Comment:
Almost every other method in ServerAmpleImpl throws RuntimeException. I will
create an issue to fix ServerAmpleImpl.
--
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]
