[GitHub] spark pull request: [SPARK-15443][SQL][Streaming] Properly explain...

2016-05-29 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/13221#issuecomment-222415753 Thanks a lot @marmbrus for your reply. My concern is that ContinuousQuery still transforms the logical plan in the runtime according to the Sources, so it is

[GitHub] spark pull request: [SPARK-15580][SQL]Add ContinuousQueryInfo to m...

2016-05-29 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/13335#discussion_r65024979 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/CompositeOffset.scala --- @@ -22,7 +22,7 @@ package

[GitHub] spark pull request: [SPARK-15580][SQL]Add ContinuousQueryInfo to m...

2016-05-29 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/13335#issuecomment-222415048 I reverted https://github.com/apache/spark/commit/c24b6b679c3efa053f7de19be73eb36dc70d9930 as there is a Option serialization bug in jackson-module-scala 2.7.3

[GitHub] spark pull request: [SPARK-15580][SQL]Add ContinuousQueryInfo to m...

2016-05-29 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/13335#discussion_r65024904 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/util/ContinuousQueryListenerSuite.scala --- @@ -183,35 +179,23 @@ class ContinuousQueryListenerSuite

[GitHub] spark pull request: [SPARK-15580][SQL]Add ContinuousQueryInfo to m...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13335#issuecomment-222414970 **[Test build #59608 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59608/consoleFull)** for PR 13335 at commit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222412709 **[Test build #59607 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59607/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-222412544 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-222412506 **[Test build #59605 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59605/consoleFull)** for PR 13219 at commit

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-222412543 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222412185 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222412184 **[Test build #59606 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59606/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222412187 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222411932 **[Test build #59606 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59606/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222409876 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222409877 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222409808 **[Test build #59604 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59604/consoleFull)** for PR 13147 at commit

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-29 Thread GayathriMurali
Github user GayathriMurali commented on the pull request: https://github.com/apache/spark/pull/13176#issuecomment-222409058 @MLnick Please let me know if there is anything else that I can help with this PR --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-222408967 **[Test build #59605 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59605/consoleFull)** for PR 13219 at commit

[GitHub] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

2016-05-29 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/13371#issuecomment-222408752 also cc @yhuai --- 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] spark pull request: [SPARK-15639][SQL] Try to push down filter at ...

2016-05-29 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/13371#issuecomment-222408282 cc @nongli @liancheng --- 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

[GitHub] spark pull request: [SPARK-14321][SQL] Reduce date format cost and...

2016-05-29 Thread rajeshbalamohan
Github user rajeshbalamohan commented on the pull request: https://github.com/apache/spark/pull/12105#issuecomment-222408035 Sorry about the delay in responding to this. Will try to rebase and post the patch asap. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-29 Thread ueshin
Github user ueshin commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222403715 @marmbrus Could you review this pr again and let me know if I have something to do to merge this. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222403702 **[Test build #59604 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59604/consoleFull)** for PR 13147 at commit

[GitHub] spark pull request: [SPARK-15647] [SQL] Fix Boundary Cases in Opti...

2016-05-29 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65019806 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -937,7 +937,8 @@ object SimplifyConditionals

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222403231 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222403233 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222403154 **[Test build #59603 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59603/consoleFull)** for PR 13351 at commit

[GitHub] spark pull request: [SPARK-14204] [SQL] register driverClass rathe...

2016-05-29 Thread zzcclp
Github user zzcclp commented on the pull request: https://github.com/apache/spark/pull/12000#issuecomment-222398286 ping @JoshRosen --- 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] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222394724 **[Test build #59603 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59603/consoleFull)** for PR 13351 at commit

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/13351#discussion_r65015447 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2345,17 +2345,19 @@ private[spark] class RedirectThread( */

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222394155 **[Test build #59602 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59602/consoleFull)** for PR 13351 at commit

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222394159 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222394158 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222394058 **[Test build #59602 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59602/consoleFull)** for PR 13351 at commit

[GitHub] spark pull request: [SPARK-15601][CORE] CircularBuffer's toString(...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/13351#discussion_r65015360 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2345,17 +2345,19 @@ private[spark] class RedirectThread( */

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13385#issuecomment-222390801 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13385#issuecomment-222390798 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread catapan
Github user catapan commented on a diff in the pull request: https://github.com/apache/spark/pull/13396#discussion_r65014383 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -54,7 +54,8 @@ function makeIdNumeric(id) { } function

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13385#issuecomment-222390746 **[Test build #59601 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59601/consoleFull)** for PR 13385 at commit

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13396#discussion_r65014109 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -54,7 +54,8 @@ function makeIdNumeric(id) { } function

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222389159 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222389158 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222389122 **[Test build #59599 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59599/consoleFull)** for PR 13396 at commit

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13395#discussion_r65013574 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -936,7 +936,47 @@ class SparkSqlAstBuilder(conf: SQLConf) extends

[GitHub] spark pull request: SPARK-15641 HistoryServer to not show invalid ...

2016-05-29 Thread catapan
Github user catapan commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222388056 revised as suggested. Thanks for the suggestions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/13395#discussion_r65013372 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -936,7 +936,47 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/13395#discussion_r65013342 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala --- @@ -936,7 +936,47 @@ class SparkSqlAstBuilder(conf: SQLConf)

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222387582 nit: the title of the PR could be made better: `[SPARK-15641] HistoryServer to not show invalid date for incomplete application` --- If your project is

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread tejasapatil
Github user tejasapatil commented on a diff in the pull request: https://github.com/apache/spark/pull/13396#discussion_r65013295 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -54,7 +54,8 @@ function makeIdNumeric(id) { }

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222386802 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222386805 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222386772 **[Test build #59600 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59600/consoleFull)** for PR 13394 at commit

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13385#issuecomment-222386241 **[Test build #59601 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59601/consoleFull)** for PR 13385 at commit

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222386047 **[Test build #59600 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59600/consoleFull)** for PR 13394 at commit

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222385824 Jenkins test this please --- 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] spark pull request: [MINOR] Resolve a number of miscellaneous buil...

2016-05-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13377 --- 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 the feature is

[GitHub] spark pull request: [MINOR] Resolve a number of miscellaneous buil...

2016-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13377#issuecomment-222384857 Merged to master/2.0 --- 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] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222384540 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222384541 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222384486 **[Test build #59598 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59598/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222384278 **[Test build #59599 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59599/consoleFull)** for PR 13396 at commit

[GitHub] spark pull request: [Docs] fix example code issues in documentatio...

2016-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13388#issuecomment-222384252 OK. Convention is to add `[MINOR]` to the title BTW. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13385#discussion_r65011967 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/receiver/Receiver.scala --- @@ -242,8 +242,7 @@ abstract class Receiver[T](val storageLevel:

[GitHub] spark pull request: [SPARK-15645][Streaming] Fix some typos of Str...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13385#discussion_r65011957 --- Diff: core/src/main/scala/org/apache/spark/rdd/ParallelCollectionRDD.scala --- @@ -32,8 +32,8 @@ import org.apache.spark.util.Utils private[spark]

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222384154 Jenkins test this please --- 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

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13396#discussion_r65011946 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -54,7 +54,8 @@ function makeIdNumeric(id) { } function

[GitHub] spark pull request: [Docs] fix example code issues in documentatio...

2016-05-29 Thread mawise
Github user mawise commented on the pull request: https://github.com/apache/spark/pull/13388#issuecomment-222383817 Nothing jumps out at me as I scan though the docs for similar errors. There might be some left- I'm not doing a super thorough review. On Sun, May 29, 2016

[GitHub] spark pull request: [SPARK-15100][ML][Doc] doc update for stopword...

2016-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/13375#issuecomment-222383405 I think that's fine. --- 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] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011713 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/MulticlassClassificationEvaluator.scala --- @@ -41,6 +41,10 @@ class

[GitHub] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011674 --- Diff: docs/mllib-evaluation-metrics.md --- @@ -182,17 +183,22 @@ $$\hat{\delta}(x) = \begin{cases}1 & \text{if $x = 0$}, \\ 0 & \text{otherwise}.

[GitHub] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011652 --- Diff: docs/mllib-evaluation-metrics.md --- @@ -136,6 +136,7 @@ for all other classes. So, a true positive occurs whenever the prediction and th

[GitHub] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011630 --- Diff: mllib/src/main/scala/org/apache/spark/ml/evaluation/MulticlassClassificationEvaluator.scala --- @@ -41,6 +41,10 @@ class

[GitHub] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011625 --- Diff: docs/mllib-evaluation-metrics.md --- @@ -182,17 +183,22 @@ $$\hat{\delta}(x) = \begin{cases}1 & \text{if $x = 0$}, \\ 0 & \text{otherwise}.

[GitHub] spark pull request: [SPARK-15617][ML][DOC] Clarify that fMeasure i...

2016-05-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13390#discussion_r65011611 --- Diff: docs/mllib-evaluation-metrics.md --- @@ -182,17 +183,22 @@ $$\hat{\delta}(x) = \begin{cases}1 & \text{if $x = 0$}, \\ 0 & \text{otherwise}.

[GitHub] spark pull request: [SPARK-14279] [Build] Pick the spark version f...

2016-05-29 Thread jodersky
Github user jodersky commented on the pull request: https://github.com/apache/spark/pull/13061#issuecomment-222380888 What I understand from the conversation is that maven assumes the target/extra-resources directory: > Unfortunately I'm not an maven expert but I wonder if we can

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222380672 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222380673 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222380607 **[Test build #59596 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59596/consoleFull)** for PR 13395 at commit

[GitHub] spark pull request: [SPARK-15646] [SQL] When spark.sql.hive.conver...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222379303 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-15646] [SQL] When spark.sql.hive.conver...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222379304 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222379286 **[Test build #59598 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59598/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-15646] [SQL] When spark.sql.hive.conver...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13386#issuecomment-222379256 **[Test build #59594 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59594/consoleFull)** for PR 13386 at commit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread NarineK
Github user NarineK commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222379022 Jenkins, retest this please. --- 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] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378750 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378755 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378756 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378743 **[Test build #59597 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59597/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378749 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378722 **[Test build #59593 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59593/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: [SPARK-15490][R][DOC] SparkR 2.0 QA: New R API...

2016-05-29 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/13394#issuecomment-222378580 Jenkins test this again please. --- 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

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222378255 **[Test build #59597 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59597/consoleFull)** for PR 12836 at commit

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13396#issuecomment-222376647 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

[GitHub] spark pull request: SPARK-15641 do not show for incomplete applica...

2016-05-29 Thread catapan
GitHub user catapan opened a pull request: https://github.com/apache/spark/pull/13396 SPARK-15641 do not show for incomplete application You can merge this pull request into a Git repository by running: $ git pull https://github.com/catapan/spark

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222376429 **[Test build #59596 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59596/consoleFull)** for PR 13395 at commit

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222376388 test this please --- 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] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222376022 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222376014 **[Test build #59595 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59595/consoleFull)** for PR 13395 at commit

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222376021 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13395#issuecomment-222375644 **[Test build #59595 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59595/consoleFull)** for PR 13395 at commit

[GitHub] spark pull request: [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS...

2016-05-29 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/13395 [SPARK-14507] [SQL] EXTERNAL keyword in a CTAS statement is not allowed ## What changes were proposed in this pull request? This PR makes the parser to throw an exception if a hive style CTAS

  1   2   >