Merge pull request #528 from ExpediaDotCom/timeline-sort-fix

sort timeline results by timestamp to fix graph bug
ExpediaDotCom · Nov 7, 2019 · 41f1ad2 · 41f1ad2
2 parents 31e5f33 + 9f829c2
commit 41f1ad2
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/server/connectors/traces/haystack/tracesConnector.js b/server/connectors/traces/haystack/tracesConnector.js
@@ -215,7 +215,8 @@ connector.getLatencyCost = (traceId) => {
 connector.getTimeline = (query) =>
     traceCountsFetcher.fetch(traceCountsRequestBuilder.buildRequest(query)).then((result) => {
         const pbTraceCounts = messages.TraceCounts.toObject(false, result);
-        return pbTraceCountsConverter.toTraceCountsJson(pbTraceCounts);
+        return pbTraceCountsConverter.toTraceCountsJson(pbTraceCounts)
+            .sort((a, b) => (a.x - b.x));
     });
 
 module.exports = connector;