Github user kkhatua commented on a diff in the pull request:
https://github.com/apache/drill/pull/954#discussion_r140957231
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/ProfileWrapper.java
---
@@ -75,24 +75,31 @@ public ProfileWrapper(final QueryProfile profile) {
final List<OperatorWrapper> ows = new ArrayList<>();
// temporary map to store (major_id, operator_id) -> [(op_profile,
minor_id)]
final Map<ImmutablePair<Integer, Integer>,
List<ImmutablePair<OperatorProfile, Integer>>> opmap = new HashMap<>();
+ // temporary map to store (major_id, operator_id) ->
[minorFragHostname]
--- End diff --
+1 on this comment. However, I didn't what to make the PR look to radically
alter what is already there. I'll incorporate the change. This should go nicely
with a fix for DRILL-5802 and make life a bit easier
---
