[
https://issues.apache.org/jira/browse/DRILL-5803?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16180247#comment-16180247
]
ASF GitHub Bot commented on DRILL-5803:
---------------------------------------
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
> Show the hostname for each minor fragment in operator table
> -----------------------------------------------------------
>
> Key: DRILL-5803
> URL: https://issues.apache.org/jira/browse/DRILL-5803
> Project: Apache Drill
> Issue Type: Sub-task
> Components: Web Server
> Reporter: Kunal Khatua
> Assignee: Kunal Khatua
> Fix For: 1.12.0
>
>
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
