[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-10-24 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 I am not sure what is my next step. And this PR was created very long time ago. So enviroment changed before. I think other contributor can resolve better than me. So I closed this PR. Thanks

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-10-15 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 Can anyone help this PR? I don't know the next step. --- 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

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-09-24 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 @Leemoonsoo When paragraph received a SIGTERM, Job result code is INCOMPLETE. Before When Job result code is INCOMPLETE, Status is FINISHED. I think in this situation user can not realize

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-09-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1350 Thanks for the contribution. @mwkang Could you elaborate little bit more how this code change is related to ZEPPELIN-1351? ZEPPELIN-1351 is about displaying paragraph status when

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-09-17 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 I added test. Please review PR, thanks for your consideration. --- 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

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread mwkang
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1350 @felixcheung Okay! I will add unit test. Thanks for your comment! --- 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

[GitHub] zeppelin issue #1350: [ZEPPELIN-1351]Paragraph status should be ERROR if Par...

2016-08-22 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/1350 is it possible to add some unit test for this? --- 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