[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [exonerator/master] Sort matches first by timestamp, then by fingerprint.



commit 805fd38d22e94205ac3d1dd204ba5e2742ba56d7
Author: Karsten Loesing <karsten.loesing@xxxxxxx>
Date:   Fri Nov 17 10:59:08 2017 +0100

    Sort matches first by timestamp, then by fingerprint.
    
    A while ago, in 92cbc07, we took out ORDER BY statements, because we
    didn't need them anymore in order to produce correct output. However,
    this also made the technical details part less readable with entries
    appearing in random order. Let's fix this by sorting results first by
    timestamp and then by fingerprint.
    
    Fixes #24327.
---
 CHANGELOG.md                                                  |  2 ++
 .../java/org/torproject/metrics/exonerator/QueryServlet.java  | 11 +++++++++++
 2 files changed, 13 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7f4cb50..aaacf94 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,8 @@
    - Rename root package org.torproject.exonerator to
      org.torproject.metrics.exonerator to make it part of the Tor
      Metrics name space.
+   - Sort results under technical details by timestamp and, if
+     necessary, by fingerprint.
 
 
 # Changes in version 2.0.0 - 2017-11-14
diff --git a/src/main/java/org/torproject/metrics/exonerator/QueryServlet.java b/src/main/java/org/torproject/metrics/exonerator/QueryServlet.java
index 2e44ed7..7406be5 100644
--- a/src/main/java/org/torproject/metrics/exonerator/QueryServlet.java
+++ b/src/main/java/org/torproject/metrics/exonerator/QueryServlet.java
@@ -18,6 +18,7 @@ import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
 import java.util.Calendar;
+import java.util.Collections;
 import java.util.List;
 import java.util.SortedSet;
 import java.util.TimeZone;
@@ -338,6 +339,16 @@ public class QueryServlet extends HttpServlet {
         }
       }
       if (!matches.isEmpty()) {
+        Collections.sort(matches,
+            (m1, m2) -> {
+              if (m1 == m2) {
+                return 0;
+              } else if (!m1.timestamp.equals(m2.timestamp)) {
+                return m1.timestamp.compareTo(m2.timestamp);
+              } else {
+                return m1.fingerprint.compareTo(m2.fingerprint);
+              }
+            });
         response.matches = matches.toArray(new QueryResponse.Match[0]);
       } else {
         List<String> nearbyAddresses = new ArrayList<>();



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits