Use builder pattern to avoid re-sorting and re-copying of bazel thread data

Signed-off-by: Felipe <afrueda97@outlook.com>

Author: afrueda97

analyzer/java/com/engflow/bazel/invocation/analyzer/bazelprofile/BazelProfile.java

@@ -42,6 +42,7 @@
 import java.util.Map;
 import java.util.Optional;
 import java.util.concurrent.atomic.AtomicLong;
+import java.util.stream.Collectors;
 import java.util.stream.Stream;
 import java.util.zip.GZIPInputStream;
 import java.util.zip.ZipException;
@@ -89,10 +90,11 @@ public static BazelProfile of(Reader reader) {
 
   private final BazelVersion bazelVersion;
   private final Map<String, String> otherData = new HashMap<>();
-  private final Map<ThreadId, ProfileThread> threads = new HashMap<>();
+  private final Map<ThreadId, ProfileThread> threads;
 
   private BazelProfile(JsonReader profileReader) {
     try {
+      Map<ThreadId, ProfileThread.Builder> threadBuilders = new HashMap<>();
       boolean hasOtherData = false;
       boolean hasTraceEvents = false;
       profileReader.beginObject();
@@ -121,17 +123,17 @@ private BazelProfile(JsonReader profileReader) {
                 continue;
               }
               ThreadId threadId = new ThreadId(pid, tid);
-              ProfileThread profileThread =
-                  threads.compute(
+              ProfileThread.Builder profileThreadBuilder =
+                  threadBuilders.compute(
                       threadId,
                       (key, t) -> {
                         if (t == null) {
-                          t = new ProfileThread(threadId);
+                          t = new ProfileThread.Builder().setThreadId(key);
                         }
                         return t;
                       });
               // TODO: Use success response to take action on errant events.
-              profileThread.addEvent(traceEvent);
+              profileThreadBuilder.addEvent(traceEvent);
             }
             profileReader.endArray();
             break;
@@ -148,6 +150,9 @@ private BazelProfile(JsonReader profileReader) {
                 TraceEventFormatConstants.SECTION_OTHER_DATA,
                 TraceEventFormatConstants.SECTION_TRACE_EVENTS));
       }
+      threads =
+          threadBuilders.entrySet().stream()
+              .collect(Collectors.toConcurrentMap(Map.Entry::getKey, e -> e.getValue().build()));
     } catch (IllegalStateException | IOException e) {
       throw new IllegalArgumentException("Could not parse Bazel profile.", e);
     }