Simplify sorting of query events result
Use a stream chain of operations and remove the now redundant method.
Change-Id: Iec01a4884d51cce958ab6a02f97e1eda36cfcb83
diff --git a/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLStore.java b/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLStore.java
index 004e82a..aa579af 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLStore.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLStore.java
@@ -15,6 +15,7 @@
package com.ericsson.gerrit.plugins.eventslog.sql;
import static com.ericsson.gerrit.plugins.eventslog.sql.SQLTable.TABLE_NAME;
+import static java.util.stream.Collectors.toList;
import com.ericsson.gerrit.plugins.eventslog.EventPool;
import com.ericsson.gerrit.plugins.eventslog.EventStore;
@@ -38,7 +39,6 @@
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.Collection;
-import java.util.Collections;
import java.util.List;
import java.util.Map.Entry;
import java.util.concurrent.ScheduledFuture;
@@ -128,16 +128,7 @@
log.warn("Cannot get project visibility info for {} from cache", projectName, e);
}
}
- return sortedEventsFromEntries(entries);
- }
-
- private List<String> sortedEventsFromEntries(List<SQLEntry> entries) {
- Collections.sort(entries);
- List<String> events = new ArrayList<>();
- for (SQLEntry entry : entries) {
- events.add(entry.getEvent());
- }
- return events;
+ return entries.stream().sorted().map(SQLEntry::getEvent).collect(toList());
}
/**
