[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
LGTM, I saw your description and it makes sense to me.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
@Leemoonsoo @jongyoul Any comments for this PR ?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
Flaky test may be another concurrency issue in scheduler execute jobs.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
I'm not sure, this PR is intent to fix the dead lock issue.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-12 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
Does this PR resolve flaky test that current master encounters?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1891: [ZEPPELIN-1941] Fix cron job with release resource opt...

2017-01-11 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1891
  
Thanks for the contribution. Can you make a test case for it?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---