[GitHub] storm pull request #2657: STORM-3048 : A Potential NPE

2018-06-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2657 ---

[GitHub] storm pull request #2657: STORM-3048 : A Potential NPE

2018-05-04 Thread Ethanlm
Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2657#discussion_r186169192 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java --- @@ -98,6 +98,7 @@ public

[GitHub] storm pull request #2657: STORM-3048 : A Potential NPE

2018-05-04 Thread Ethanlm
Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2657#discussion_r186169251 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java --- @@ -135,6 +136,7 @@

[GitHub] storm pull request #2657: STORM-3048 : A Potential NPE

2018-05-04 Thread Ethanlm
Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2657#discussion_r186173164 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java --- @@ -98,6 +98,7 @@ public

[GitHub] storm pull request #2657: STORM-3048 : A Potential NPE

2018-05-04 Thread Ethanlm
Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2657#discussion_r186169276 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/ConstraintSolverStrategy.java --- @@ -173,6 +175,7 @@