[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

2016-05-17 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/8559#issuecomment-219840630 @mateiz @markhamstra @kayousterhout I've updated this to avoid changing the scheduler at all, and instead just run the mock backend in another thread. It slightly

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13151#issuecomment-219840935 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-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

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

[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

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

[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8559#issuecomment-219841367 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-15362] [ML] Make spark.ml KMeansModel l...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13149#discussion_r63598420 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala --- @@ -206,11 +212,21 @@ object KMeansModel extends MLReadable[KMeansModel] {

[GitHub] spark pull request: [SPARK-15362] [ML] Make spark.ml KMeansModel l...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13149#discussion_r63598417 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala --- @@ -206,11 +212,21 @@ object KMeansModel extends MLReadable[KMeansModel] {

[GitHub] spark pull request: [SPARK-15362] [ML] Make spark.ml KMeansModel l...

2016-05-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/13149#issuecomment-219845117 Thanks! I should have done this before. --- 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-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63599112 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15361] [ML] ML 2.0 QA: Scala APIs audit...

2016-05-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/13148#issuecomment-219846049 I'll take a look now --- 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-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13153#issuecomment-219848240 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-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15361] [ML] ML 2.0 QA: Scala APIs audit...

2016-05-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/13148#issuecomment-219848715 That's the only issue I saw. Thanks! --- 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

[GitHub] spark pull request: [SPARK-15361] [ML] ML 2.0 QA: Scala APIs audit...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13148#discussion_r63600571 --- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/BisectingKMeans.scala --- @@ -39,23 +39,27 @@ private[clustering] trait

[GitHub] spark pull request: [SPARK-14400] [SQL] ScriptTransformation does ...

2016-05-17 Thread tejasapatil
Github user tejasapatil commented on the pull request: https://github.com/apache/spark/pull/12194#issuecomment-219853586 ok to test --- 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-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63603796 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -451,13 +462,15 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63604166 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-14400] [SQL] ScriptTransformation does ...

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

[GitHub] spark pull request: [SPARK-8674] [MLlib] Implementation of a 2 sam...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7075#issuecomment-219855097 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 project does

[GitHub] spark pull request: [SPARK-8674] [MLlib] Implementation of a 2 sam...

2016-05-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7075#issuecomment-219854991 [Test build #58715 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58715/console) for PR 7075 at commit

[GitHub] spark pull request: [SPARK-8674] [MLlib] Implementation of a 2 sam...

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

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-17 Thread frreiss
GitHub user frreiss opened a pull request: https://github.com/apache/spark/pull/13155 [SPARK-15370] [SQL] Update RewriteCorrelatedScalarSubquery rule to fix COUNT bug ## What changes were proposed in this pull request? This pull request fixes the COUNT bug in the

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13155#issuecomment-21986 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-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63605320 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -348,13 +349,23 @@ object

[GitHub] spark pull request: [SPARK-15269][SQL] Set provided path to Catalo...

2016-05-17 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/13120#discussion_r63605497 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -451,13 +462,15 @@ object

[GitHub] spark pull request: Branch 2.0

2016-05-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/13089#issuecomment-219859010 Is this PR a mistake (with no title, linked JIRA, description)? Can you please close this issue? --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [WIP]Check SparkContext not stop

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

2016-05-17 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/10125#discussion_r63608338 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystTypeConverters.scala --- @@ -321,11 +323,13 @@ object CatalystTypeConverters

[GitHub] spark pull request: [SPARK-11735][Core][SQL]Add a check in the con...

2016-05-17 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/13154#issuecomment-219861724 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 feature

[GitHub] spark pull request: [SPARK-11735][Core][SQL]Add a check in the con...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12719#issuecomment-219864350 Rebased to resolve conflicts. --- 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

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

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

[GitHub] spark pull request: [SPARK-14400] [SQL] ScriptTransformation does ...

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

[GitHub] spark pull request: [SPARK-14400] [SQL] ScriptTransformation does ...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12194#issuecomment-219865151 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-14400] [SQL] ScriptTransformation does ...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11735][Core][SQL]Add a check in the con...

2016-05-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13154#issuecomment-219866146 LGTM. Merging to master and branch 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

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

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

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

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

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12719#issuecomment-219866626 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-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11735][Core][SQL]Add a check in the con...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13154 --- 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: [SPARK-15367] [SQL] Add refreshTable back

2016-05-17 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/13156 [SPARK-15367] [SQL] Add refreshTable back What changes were proposed in this pull request? `refreshTable` was a method in `HiveContext`. It was deleted accidentally while we were

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

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

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13156#discussion_r63613130 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -294,6 +294,18 @@ class SQLContext private[sql](

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13156#discussion_r63613203 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/MultiDatabaseSuite.scala --- @@ -202,7 +202,8 @@ class MultiDatabaseSuite extends QueryTest with

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13153#issuecomment-219870033 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-15317][Core]Don't store accumulators fo...

2016-05-17 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/13153#issuecomment-219870943 cc @andrewor14 @yhuai @jkbradley --- 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-15367] [SQL] Add refreshTable back

2016-05-17 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13156#discussion_r63615033 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -294,6 +294,18 @@ class SQLContext private[sql](

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

2016-05-17 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13156#discussion_r63615059 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/MultiDatabaseSuite.scala --- @@ -202,7 +202,8 @@ class MultiDatabaseSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13151#issuecomment-219874513 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-15031][EXAMPLES][FOLLOW-UP] Make Python...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/13135#issuecomment-219876620 @MLnick Yes. It seems `threshold` and `thresholds` are set exclusively via `set..` methods but in Python both can be set via `fit` method. --- If your project is

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

2016-05-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13153#issuecomment-219877343 Maybe also put the size comparison in the description? --- 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-14346][SQL] Native SHOW CREATE TABLE fo...

2016-05-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13079#issuecomment-219878464 LGTM. I am merging this to master and 2.0. Let's change the error message and exception class in a separate pr. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-14346][SQL] Native SHOW CREATE TABLE fo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13079#discussion_r63618404 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala --- @@ -626,40 +626,142 @@ case class ShowCreateTableCommand(table:

[GitHub] spark pull request: [SPARK-14346][SQL] Native SHOW CREATE TABLE fo...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13079 --- 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: [SPARK-1301] [Web UI] Added anchor links to Ac...

2016-05-17 Thread ajbozarth
Github user ajbozarth commented on the pull request: https://github.com/apache/spark/pull/13037#issuecomment-219881338 @srowen @andrewor14 I agree that though is not the last straw, the vertically growing list is a issue. I'll throw a fix together with a table view and post a screen

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12719#issuecomment-219881850 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 have

[GitHub] spark pull request: [SPARK-14346][SQL] Native SHOW CREATE TABLE fo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13079#discussion_r63620803 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -200,6 +207,7 @@ case class SimpleCatalogRelation(

[GitHub] spark pull request: [SPARK-14939][SQL] Add FoldablePropagation opt...

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

[GitHub] spark pull request: [SPARK-14346] Fix scala-2.10 build

2016-05-17 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/13157 [SPARK-14346] Fix scala-2.10 build ## What changes were proposed in this pull request? Scala 2.10 build was broken by https://github.com/apache/spark/pull/13079/files. I am reverting the change

[GitHub] spark pull request: [SPARK-14346] Fix scala-2.10 build

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

[GitHub] spark pull request: [SPARK-11735][Core][SQL]Add a check in the con...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10125#issuecomment-219885198 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 project does

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-14346] Fix scala-2.10 build

2016-05-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13157#issuecomment-219885469 tested locally. The fix is good. --- 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-15367] [SQL] Add refreshTable back

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

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

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

[GitHub] spark pull request: [SPARK-15367] [SQL] Add refreshTable back

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13156#issuecomment-219886319 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-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10125#issuecomment-219888687 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 project does

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

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

[GitHub] spark pull request: [SPARK-11827] [SQL] Adding java.math.BigIntege...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10125#issuecomment-219888712 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 project does

[GitHub] spark pull request: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow se...

2016-05-17 Thread zjffdu
Github user zjffdu commented on a diff in the pull request: https://github.com/apache/spark/pull/13146#discussion_r63625742 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -62,7 +62,8 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with Logging {

[GitHub] spark pull request: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow se...

2016-05-17 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/spark/pull/13146#issuecomment-219890162 @BryanCutler Thanks for review. The reason I create this ticket is that it is not convenient for user to specify environment variable especially when user want to

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

2016-05-17 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/12000#issuecomment-219890576 I'm on vacation right now. This patch seems pretty small and reasonable to me, though, so it would be great if someone else could take a final look and merge if

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13155#discussion_r63626483 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala --- @@ -293,4 +293,65 @@ class SubquerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-15370] [SQL] Update RewriteCorrelatedSc...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13155#discussion_r63626503 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala --- @@ -293,4 +293,65 @@ class SubquerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13150#discussion_r63626710 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala --- @@ -114,17 +115,27 @@ private[hive] case class

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13150#discussion_r63626678 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala --- @@ -114,17 +115,27 @@ private[hive] case class

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/spark/pull/13150#discussion_r63627480 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala --- @@ -114,17 +115,27 @@ private[hive] case class

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread Parth-Brahmbhatt
Github user Parth-Brahmbhatt commented on a diff in the pull request: https://github.com/apache/spark/pull/13150#discussion_r63627565 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala --- @@ -114,17 +115,27 @@ private[hive] case class

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/13150#discussion_r63628099 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/MetastoreRelation.scala --- @@ -114,17 +115,27 @@ private[hive] case class

[GitHub] spark pull request: [SPARK-15373][WEB UI] Spark UI should show con...

2016-05-17 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/13158 [SPARK-15373][WEB UI] Spark UI should show consistent timezones. ## What changes were proposed in this pull request? Currently, SparkUI shows two timezones in a single page when the

[GitHub] spark pull request: [DOC][MINOR] ml.feature Scala and Python API s...

2016-05-17 Thread BryanCutler
Github user BryanCutler commented on the pull request: https://github.com/apache/spark/pull/13159#issuecomment-219893757 cc @MLnick @holdenk --- 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: [DOC][MINOR] ml.feature Scala and Python API s...

2016-05-17 Thread BryanCutler
GitHub user BryanCutler opened a pull request: https://github.com/apache/spark/pull/13159 [DOC][MINOR] ml.feature Scala and Python API sync ## What changes were proposed in this pull request? I reviewed Scala and Python APIs for ml.feature and corrected discrepancies.

[GitHub] spark pull request: [SPARK-15373][WEB UI] Spark UI should show con...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13158#issuecomment-219893875 cc @davies --- 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: [DOC][MINOR] ml.feature Scala and Python API s...

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

<    1   2   3   4   5   6   >