commit 43a330435ffab9e3bd84b15b83d16d9c6de87984
author Hector Oswaldo Caballero <hector.caballero@ericsson.com> Sat Jul 14 07:55:21 2018 -0400
committer Hector Oswaldo Caballero <hector.caballero@ericsson.com> Tue Jul 17 05:57:25 2018 -0400
tree 83a16078749b94cd53e363167afb4d0a068477ec
parent 8864adc111a24a4223d85551f6eb032e15adc99c

Make default query more readable

Change-Id: I1bfe7fe244d7b8ac1055739636f4b37b06c8546d

src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLQueryMaker.java

diff --git a/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLQueryMaker.java b/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLQueryMaker.java
index 53e406c..c84bfc9 100644
--- a/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLQueryMaker.java
+++ b/src/main/java/com/ericsson/gerrit/plugins/eventslog/sql/SQLQueryMaker.java
@@ -65,16 +65,9 @@
 
   @Override
   public String getDefaultQuery() {
-    return "SELECT * FROM(SELECT * FROM "
-        + TABLE_NAME
-        + " ORDER BY "
-        + PRIMARY_ENTRY
-        + " DESC LIMIT "
-        + returnLimit
-        + ")"
-        + " a ORDER BY "
-        + PRIMARY_ENTRY
-        + " ASC";
+    return String.format(
+        "SELECT * FROM (SELECT * FROM %s ORDER BY %s DESC LIMIT %s) a ORDER BY %s ASC",
+        TABLE_NAME, PRIMARY_ENTRY, returnLimit, PRIMARY_ENTRY);
   }
 
   private String[] parseDates(String dateOne, String dateTwo)