[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-16 Thread benoyantony
Github user benoyantony commented on the issue: https://github.com/apache/zeppelin/pull/2223 Thanks @zjffdu for reviewing and committing. --- 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

[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-14 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2223 Thanks @benoyantony will merge it if no more comments. --- 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 #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-10 Thread benoyantony
Github user benoyantony commented on the issue: https://github.com/apache/zeppelin/pull/2223 A testcase errored out and it seems to be unrelated. ParagraphActionsIT.testRunOnSelectionChange --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2223 @benoyantony You can call cancel in a separate thread, the same as testSparkInterpreterRDD, let me know if you have any other questions. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread benoyantony
Github user benoyantony commented on the issue: https://github.com/apache/zeppelin/pull/2223 @zjffdu , Will try to add the unit test for cancellation in LivySparkSQLInterpreter. Any suggestion on how to simulate a sleep in SQL ? Also, where are the other comments ? Sorry, I am

[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2223 @benoyantony Thanks for the contribution, just left some comments. --- 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 #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread benoyantony
Github user benoyantony commented on the issue: https://github.com/apache/zeppelin/pull/2223 @zjffdu , Could you please review ? --- 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