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 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 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 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 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 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 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 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