[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332659487 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskDriver.java ## @@ -388,11 +389,19 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332659308 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -989,47 +988,65 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332659068 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -989,47 +988,65 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r33265 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -989,47 +988,65 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332658821 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -717,41 +717,40 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332658622 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskDriver.java ## @@ -388,11 +389,19 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332658421 ## File path: helix-core/src/test/java/org/apache/helix/integration/task/TestPurgeJobWithoutConfig.java

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-08 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r332658541 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskDriver.java ## @@ -388,11 +389,19 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331722786 ## File path: helix-core/src/test/java/org/apache/helix/integration/task/TestPurgeJobWithoutConfig.java

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331720377 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -990,46 +1009,65 @@

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331722875 ## File path: helix-core/src/test/java/org/apache/helix/integration/task/TestPurgeJobWithoutConfig.java

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331722856 ## File path: helix-core/src/test/java/org/apache/helix/integration/task/TestPurgeJobWithoutConfig.java

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331722786 ## File path: helix-core/src/test/java/org/apache/helix/integration/task/TestPurgeJobWithoutConfig.java

[GitHub] [helix] alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config

2019-10-04 Thread GitBox
alirezazamani commented on a change in pull request #497: Fix job purge logic for job without config URL: https://github.com/apache/helix/pull/497#discussion_r331720377 ## File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java ## @@ -990,46 +1009,65 @@