diff options
author | Sébastien Lesaint <sebastien.lesaint@sonarsource.com> | 2016-06-01 13:52:33 +0200 |
---|---|---|
committer | Sébastien Lesaint <sebastien.lesaint@sonarsource.com> | 2016-06-02 14:06:05 +0200 |
commit | 83227c7b26c855242959c099ee868d911f330357 (patch) | |
tree | 4706f7188146d2c8448079e2ab8b00c2340aadf2 /sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java | |
parent | 4d6001278cc4d304b59bc281142238df503425a7 (diff) | |
download | sonarqube-83227c7b26c855242959c099ee868d911f330357.tar.gz sonarqube-83227c7b26c855242959c099ee868d911f330357.zip |
fix quality flaws: use lambdas with DatabaseUtils.executeLargeInputs
Diffstat (limited to 'sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java')
-rw-r--r-- | sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java b/sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java index 7e62eabfc55..2f1ca4fbc7d 100644 --- a/sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java +++ b/sonar-db/src/main/java/org/sonar/db/purge/PurgeCommands.java @@ -20,13 +20,11 @@ package org.sonar.db.purge; import com.google.common.annotations.VisibleForTesting; -import com.google.common.base.Function; import com.google.common.collect.Iterables; import com.google.common.collect.Lists; import com.google.common.collect.Sets; import java.util.LinkedHashSet; import java.util.List; -import javax.annotation.Nullable; import org.apache.ibatis.session.SqlSession; import static com.google.common.collect.FluentIterable.from; @@ -40,13 +38,6 @@ class PurgeCommands { private final SqlSession session; private final PurgeMapper purgeMapper; private final PurgeProfiler profiler; - private final Function<PurgeSnapshotQuery, Iterable<Long>> purgeSnapshotQueryToSnapshotIds = new Function<PurgeSnapshotQuery, Iterable<Long>>() { - @Nullable - @Override - public Iterable<Long> apply(PurgeSnapshotQuery query) { - return purgeMapper.selectSnapshotIds(query); - } - }; PurgeCommands(SqlSession session, PurgeMapper purgeMapper, PurgeProfiler profiler) { this.session = session; @@ -156,7 +147,8 @@ class PurgeCommands { void deleteSnapshots(PurgeSnapshotQuery... queries) { List<Long> snapshotIds = from(asList(queries)) - .transformAndConcat(purgeSnapshotQueryToSnapshotIds).toList(); + .transformAndConcat(purgeMapper::selectSnapshotIds) + .toList(); deleteSnapshots(snapshotIds); } @@ -191,7 +183,7 @@ class PurgeCommands { void purgeSnapshots(PurgeSnapshotQuery... queries) { // use LinkedHashSet to keep order by remove duplicated ids - LinkedHashSet<Long> snapshotIds = Sets.newLinkedHashSet(from(asList(queries)).transformAndConcat(purgeSnapshotQueryToSnapshotIds)); + LinkedHashSet<Long> snapshotIds = Sets.newLinkedHashSet(from(asList(queries)).transformAndConcat(purgeMapper::selectSnapshotIds)); purgeSnapshots(snapshotIds); } |