simpler remove logic

elastic · May 10, 2017 · c37d9b8 · c37d9b8
1 parent 9e29820
commit c37d9b8
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/...ava/org/elasticsearch/action/admin/cluster/snapshots/get/TransportGetSnapshotsAction.java b/...ava/org/elasticsearch/action/admin/cluster/snapshots/get/TransportGetSnapshotsAction.java
@@ -164,9 +164,8 @@ private List<SnapshotInfo> buildSimpleSnapshotInfos(final Set<SnapshotId> toReso
                                                         final List<SnapshotInfo> currentSnapshots) {
         List<SnapshotInfo> snapshotInfos = new ArrayList<>();
         for (SnapshotInfo snapshotInfo : currentSnapshots) {
-            if (toResolve.contains(snapshotInfo.snapshotId())) {
+            if (toResolve.remove(snapshotInfo.snapshotId())) {
                 snapshotInfos.add(snapshotInfo.basic());
-                toResolve.remove(snapshotInfo.snapshotId());
             }
         }
         Map<SnapshotId, List<String>> snapshotsToIndices = new HashMap<>();