[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1141 ---

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread kkhatua
Github user kkhatua commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171767495 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats stats)

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171753536 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats st

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread kkhatua
Github user kkhatua commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171748860 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats stats)

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread kkhatua
Github user kkhatua commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171740245 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats stats)

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171723902 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats st

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread kkhatua
Github user kkhatua commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171648058 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats stats)

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171611927 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -79,4 +71,21 @@ public void addOperatorStats(OperatorStats st

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-03-01 Thread kkhatua
Github user kkhatua commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171485412 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -31,6 +32,13 @@ public class FragmentStats { // private

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-02-28 Thread amansinha100
Github user amansinha100 commented on a diff in the pull request: https://github.com/apache/drill/pull/1141#discussion_r171431227 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentStats.java --- @@ -31,6 +32,13 @@ public class FragmentStats { // pr

[GitHub] drill pull request #1141: DRILL-6197: Skip duplicate entry for OperatorStats

2018-02-28 Thread kkhatua
GitHub user kkhatua opened a pull request: https://github.com/apache/drill/pull/1141 DRILL-6197: Skip duplicate entry for OperatorStats `org.apache.drill.exec.ops.FragmentStats` should skip injecting the `org.apache.drill.exec.ops.OperatorStats` instance for these operators: ```