[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-20 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/16644 That code is fine and compiles correctly in master, 2.1, and 2.0. See Jenkins builds. Please close the PR. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-20 Thread ganeshkrishnan1
Github user ganeshkrishnan1 commented on the issue: https://github.com/apache/spark/pull/16644 Error:(194, 8) method sparkContext in class Builder cannot be accessed in org.apache.spark.sql.SparkSession.Builder .sparkContext(sc) Using Spark 2.1.0 and scala 2.11

[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/16644 Yes it is. What error do you get? these examples are compiled with the code and compiles correctly in master and previous 2.x builds. --- If your project is set up for it, you can reply to this

[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-19 Thread ganeshkrishnan1
Github user ganeshkrishnan1 commented on the issue: https://github.com/apache/spark/pull/16644 Fails to compile on Spark 2.1.0 Shouldn't mlib be backward compatible? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-19 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/16644 Please see http://spark.apache.org/contributing.html first This is an example for .mllib. While it works with 2.x, or should, it is not intended to use SparkSession, as I understand. ---

[GitHub] spark issue #16644: Update LDAExample with Spark 2.1.0

2017-01-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16644 Can one of the admins verify this patch? --- 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