[accumulo] branch 1451-external-compactions-feature updated: re #1999 - use copy of selectedfiles to ensure that it doesn't change

[dlmarion]

This is an automated email from the ASF dual-hosted git repository.

dlmarion pushed a commit to branch 1451-external-compactions-feature
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to 
refs/heads/1451-external-compactions-feature by this push:
     new 676b0ab  re #1999 - use copy of selectedfiles to ensure that it 
doesn't change
676b0ab is described below

commit 676b0ab3cb57089f8e74afbe7f7cc37a1e299890
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Tue Apr 13 15:13:28 2021 +0000

    re #1999 - use copy of selectedfiles to ensure that it doesn't change
---
 .../main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
index 18de722..0bcb561 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
@@ -523,8 +523,7 @@ public class CompactableImpl implements Compactable {
                 Map<String,String> hints = Map.of();
                 if (kind == CompactionKind.USER)
                   hints = compactionConfig.getExecutionHints();
-                return Optional.of(new Compactable.Files(files, 
Set.copyOf(selectedFiles),
-                    runningJobsCopy, hints));
+                return Optional.of(new Compactable.Files(files, candidates, 
runningJobsCopy, hints));
               } else {
                 return Optional.of(new Compactable.Files(files, Set.of(), 
runningJobsCopy));
               }