GitHub user kkhatua opened a pull request:
https://github.com/apache/drill/pull/742
DRILL-5242: The UI breaks when rendering profiles having unknown metrics
Skip any metrics whose metric ID is unknown, This prevents any
ArrayIndexOutOfBoundsException from being thrown and breaking t
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/742#discussion_r99684283
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java
---
@@ -163,11 +165,18 @@ public String getMetricsTa
Github user kkhatua commented on a diff in the pull request:
https://github.com/apache/drill/pull/742#discussion_r99975093
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java
---
@@ -163,11 +165,18 @@ public String getMetricsTable(
Github user kkhatua commented on a diff in the pull request:
https://github.com/apache/drill/pull/742#discussion_r100612470
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java
---
@@ -163,11 +165,18 @@ public String getMetricsTable
Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/742#discussion_r101440816
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/OperatorWrapper.java
---
@@ -163,11 +165,18 @@ public String getMetricsT
Github user kkhatua closed the pull request at:
https://github.com/apache/drill/pull/742
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab