[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1195 CI is green. Merge it to master and branch-0.6 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. I

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-22 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1195 Changes looks good. Could you rebase to master? --- 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 #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-20 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1195 Tested this with spark 2.0 and scala 2.11. Spark 2.0 works well with scala, R and python. @minahlee comment makes sense - we should ship the latest version. --- If your project is se

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-19 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1195 After I look close the pom files, I don't see clear reason why spark.version is specified in scala-2.11 and scala-2.10 profiles. https://github.com/apache/zeppelin/blob/master/spark-dependenc

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1195 @minahlee I was trying to make 'zeppelin-spark' interpreter support spark 2.0 in this PR. If there're user demands, spark 2.0 support in 'zeppelin-zrinterpreter' can be addressed in separate is

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-19 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1195 Shall we also change spark version in https://github.com/apache/zeppelin/blob/master/r/pom.xml#L386 and https://github.com/apache/zeppelin/blob/master/r/pom.xml#L398 ? --- If your project is s

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-18 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1195 Looks great 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 i

[GitHub] zeppelin issue #1195: [ZEPPELIN-759] Spark 2.0 support

2016-07-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1195 Now CI is green. Review this please. CI profile for spark 2.0 force spark version `2.0.0-preview` instead of `2.0.0`, because of spark 2.0.0 is not yet released. If this PR merges befor