[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread cloverhearts
Github user cloverhearts commented on the issue: https://github.com/apache/zeppelin/pull/1877 LGTM! --- 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

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1877 LGTM and merge to master if there're no more discussions --- 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 #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1877 LGTM! --- 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

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1877 @Leemoonsoo I added testcase, could you help to 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 h

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1877 @agoodm pyspark interpreter is working well. ![image](https://cloud.githubusercontent.com/assets/3348133/21768320/e79bebe0-d62d-11e6-8878-3c837fb7938f.png) --- If your project is set up fo

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1877 I faced the same problem too. Thanks for fix! --- 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 feat

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread agoodm
Github user agoodm commented on the issue: https://github.com/apache/zeppelin/pull/1877 I am on vacation right now so I can't thoroughly examine this, but have you confirmed that this isn't a problem for the pyspark interpreter? --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1877: [ZEPPELIN-1930](HotFix) PythonInterpreter syntax error...

2017-01-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1877 Thanks for the fix. \cc @agoodm How about add an unittest for this case in https://github.com/apache/zeppelin/blob/master/python/src/test/java/org/apache/zeppelin/python/PythonInterpre