[GitHub] spark pull request: [SPARK-15648][SQL]add TeradataDialect.scala

2016-05-30 Thread lihongliustc
Github user lihongliustc commented on the pull request: https://github.com/apache/spark/pull/13359#issuecomment-222607185 @srowen Hi,srowen,does the PR be delayed?What should I do to it? --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

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

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

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222604484 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222604316 **[Test build #59636 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59636/consoleFull)** for PR 13392 at commit [`b2849e8`](https://g

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

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

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222603353 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222603171 **[Test build #59635 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59635/consoleFull)** for PR 13270 at commit [`336fb55`](https://g

[GitHub] spark pull request: [SPARK-15644] [MLlib] [SQL] Replace SQLContext...

2016-05-30 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13380#issuecomment-222602457 cc @jkbradley / @mengxr are we ok with changing the API? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

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

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13402#issuecomment-222601873 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222601745 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

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

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13402#issuecomment-222601649 **[Test build #59633 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59633/consoleFull)** for PR 13402 at commit [`6d614dd`](https://g

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

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222601564 **[Test build #59634 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59634/consoleFull)** for PR 13392 at commit [`4306c4f`](https://g

[GitHub] spark pull request: [SPARK-15076][SQL] Improve ConstantFolding opt...

2016-05-30 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/12850#discussion_r65127553 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -751,6 +751,16 @@ object ConstantFolding extends

[GitHub] spark pull request: [SPARK-15659][SQL] Ensure FileSystem is gotten...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13405#issuecomment-222600020 **[Test build #59640 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59640/consoleFull)** for PR 13405 at commit [`7e01426`](https://gi

[GitHub] spark pull request: [SPARK-15659][SQL] Ensure FileSystem is gotten...

2016-05-30 Thread jerryshao
GitHub user jerryshao opened a pull request: https://github.com/apache/spark/pull/13405 [SPARK-15659][SQL] Ensure FileSystem is gotten from path ## What changes were proposed in this pull request? Currently `spark.sql.warehouse.dir` is pointed to local dir by default, which

[GitHub] spark pull request: [SPARK-15076][SQL] Improve ConstantFolding opt...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/12850#discussion_r65126741 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -751,6 +751,16 @@ object ConstantFolding extends Rule

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13370 --- 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 ena

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13404#issuecomment-222597618 **[Test build #59639 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59639/consoleFull)** for PR 13404 at commit [`d94b2f6`](https://gi

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13370#issuecomment-222597634 Thanks - merging in 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 h

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r65126050 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala --- @@ -204,8 +205,8 @@ class KeyValueGroupedDataset[K, V] private[sql](

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

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

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13404#issuecomment-222597425 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13404#issuecomment-222597411 **[Test build #59638 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59638/consoleFull)** for PR 13404 at commit [`94f666a`](https://g

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

2016-05-30 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13404#discussion_r65125923 --- Diff: core/src/main/java/org/apache/spark/api/java/function/package.scala --- @@ -22,4 +22,5 @@ package org.apache.spark.api.java * these int

[GitHub] spark pull request: [MINOR][CORE][DOCS] Fix description of FilterF...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13404#issuecomment-222596540 **[Test build #59638 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59638/consoleFull)** for PR 13404 at commit [`94f666a`](https://gi

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

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

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222596478 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [MINOR][CORE][DOC Fix description of FilterFun...

2016-05-30 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/13404 [MINOR][CORE][DOC Fix description of FilterFunction ## What changes were proposed in this pull request? This PR fixes the wrong description of `FilterFunction`. ``` - * If the

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

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

[GitHub] spark pull request: [SPARK-15557][SQL] expressi[on ((cast(99 as de...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13368#issuecomment-222596139 cc @yhuai @davies , do you still remember why we promote string to decimal(38, 18) instead of double? --- If your project is set up for it, you can reply to this ema

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-222595900 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

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

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

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

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13370#issuecomment-222595668 LGTM --- 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 enable

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r65125196 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/KeyValueGroupedDataset.scala --- @@ -204,8 +205,8 @@ class KeyValueGroupedDataset[K, V] private[sql]

[GitHub] spark pull request: [SPARK-15660][CORE] RDD and Dataset should sho...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13403#issuecomment-222595435 **[Test build #59637 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59637/consoleFull)** for PR 13403 at commit [`3fe0cb6`](https://gi

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r65125156 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Column.scala --- @@ -132,6 +130,15 @@ class Column(protected[sql] val expr: Expression) extends Logg

[GitHub] spark pull request: [SPARK-15638][SQL] Audit Dataset, SparkSession...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13370#discussion_r65125096 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Column.scala --- @@ -132,6 +130,15 @@ class Column(protected[sql] val expr: Expression) extends Logg

[GitHub] spark pull request: [SPARK-15660][CORE] RDD and Dataset should sho...

2016-05-30 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/13403 [SPARK-15660][CORE] RDD and Dataset should show the consistent values for variance/stdev. ## What changes were proposed in this pull request? In Spark-11490, `variance/stdev` are red

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

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-22259 **[Test build #59636 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59636/consoleFull)** for PR 13392 at commit [`b2849e8`](https://gi

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

2016-05-30 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65124178 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -219,4 +220,41 @@ class SQLConfSuite extends QueryTest with Sha

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

2016-05-30 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65124192 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -219,4 +220,41 @@ class SQLConfSuite extends QueryTest with Sha

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

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

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13401#issuecomment-222593250 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13401#issuecomment-222593159 **[Test build #59629 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59629/consoleFull)** for PR 13401 at commit [`b6c1a5f`](https://g

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222592978 **[Test build #59635 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59635/consoleFull)** for PR 13270 at commit [`336fb55`](https://gi

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222592731 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

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

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

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

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65123693 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -219,4 +220,41 @@ class SQLConfSuite extends QueryTest with Shar

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222592448 LGTM except some style comments --- 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 n

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222592244 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

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

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65123624 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -219,4 +220,41 @@ class SQLConfSuite extends QueryTest with Shar

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222592234 **[Test build #59632 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59632/consoleFull)** for PR 13270 at commit [`3830dbb`](https://g

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13402#issuecomment-222592066 cc @yhuai @zsxwing --- 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-15647] [SQL] Fix Boundary Cases in Opti...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222592083 **[Test build #59634 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59634/consoleFull)** for PR 13392 at commit [`4306c4f`](https://gi

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13402#issuecomment-222592077 **[Test build #59633 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59633/consoleFull)** for PR 13402 at commit [`6d614dd`](https://gi

[GitHub] spark pull request: [SPARK-15658][SQL] UDT serializer should decla...

2016-05-30 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/13402 [SPARK-15658][SQL] UDT serializer should declare its data type as udt instead of udt.sqlType ## What changes were proposed in this pull request? When we build serializer for UDT object,

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13270#issuecomment-222591642 **[Test build #59632 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59632/consoleFull)** for PR 13270 at commit [`3830dbb`](https://gi

[GitHub] spark pull request: [SPARK-15269][SQL] Removes unexpected empty ta...

2016-05-30 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/13270#discussion_r65123201 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala --- @@ -212,11 +212,46 @@ class SessionCatalog( *

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

2016-05-30 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65122900 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -17,13 +17,36 @@ package org.apache.spark.sql.interna

[GitHub] spark pull request: [SPARK-14649][CORE] DagScheduler should not ru...

2016-05-30 Thread sitalkedia
Github user sitalkedia commented on the pull request: https://github.com/apache/spark/pull/12436#issuecomment-222590636 @kayousterhout - Sure, I will resolve the conflicts. Can you take a cursory look at the diff and let me know if the approach is reasonable? --- If your project is

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65122821 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -17,13 +17,36 @@ package org.apache.spark.sql.internal

[GitHub] spark pull request: [SPARK-10530] [CORE] Kill other task attempts ...

2016-05-30 Thread devaraj-kavali
Github user devaraj-kavali commented on the pull request: https://github.com/apache/spark/pull/11996#issuecomment-222589679 Thanks @kayousterhout. --- 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 h

[GitHub] spark pull request: [SPARK-15655] [SQL] Fix Wrong Partition Column...

2016-05-30 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13400#discussion_r65122095 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1537,6 +1537,35 @@ class SQLQuerySuite extends QueryTes

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

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

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

2016-05-30 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65121902 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -17,13 +17,36 @@ package org.apache.spark.sql.interna

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

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

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120888 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

[GitHub] spark pull request: [SPARK-15655] [SQL] Fix Wrong Partition Column...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13400#discussion_r65120902 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1537,6 +1537,35 @@ class SQLQuerySuite extends QueryTest

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120891 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65120800 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/internal/SQLConfSuite.scala --- @@ -17,13 +17,36 @@ package org.apache.spark.sql.internal

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120675 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120665 --- Diff: R/pkg/R/deserialize.R --- @@ -197,6 +197,31 @@ readMultipleObjects <- function(inputCon) { data # this is a list of named lists now }

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120671 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120653 --- Diff: R/pkg/R/group.R --- @@ -142,3 +142,54 @@ createMethods <- function() { } createMethods() + +#' gapply +#' +#' Applie

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120620 --- Diff: R/pkg/R/group.R --- @@ -142,3 +142,54 @@ createMethods <- function() { } createMethods() + +#' gapply +#' +#' Applie

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120599 --- Diff: R/pkg/inst/worker/worker.R --- @@ -84,67 +84,78 @@ broadcastElap <- elapsedSecs() # as number of partitions to create. numPartitions <- Sp

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120565 --- Diff: R/pkg/inst/worker/worker.R --- @@ -84,67 +84,78 @@ broadcastElap <- elapsedSecs() # as number of partitions to create. numPartitions <- Sp

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120457 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala --- @@ -379,6 +383,50 @@ class RelationalGroupedDataset protected[sql](

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120461 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -2011,6 +2011,25 @@ class Dataset[T] private[sql]( } /** +

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

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/13392#discussion_r65120442 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -937,9 +937,14 @@ object SimplifyConditionals extends

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120450 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/objects.scala --- @@ -325,6 +330,80 @@ case class MapGroupsExec( } /** +

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

2016-05-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13401#issuecomment-222586062 **[Test build #59629 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59629/consoleFull)** for PR 13401 at commit [`b6c1a5f`](https://gi

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

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

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120403 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

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

2016-05-30 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12836#discussion_r65120399 --- Diff: R/pkg/R/DataFrame.R --- @@ -1268,6 +1268,82 @@ setMethod("dapplyCollect", ldf }) +#' gapply +#' +#

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

2016-05-30 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/13401#issuecomment-222585795 cc @marmbrus @yhuai @viirya --- 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 no

[GitHub] spark pull request: [SPARK-15657][SQL] RowEncoder should validate ...

2016-05-30 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/13401 [SPARK-15657][SQL] RowEncoder should validate the data type of input object ## What changes were proposed in this pull request? This PR improves the error handling of `RowEncoder`. When w

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13147#issuecomment-222585528 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

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

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

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

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

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

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

2016-05-30 Thread tejasapatil
Github user tejasapatil commented on the pull request: https://github.com/apache/spark/pull/13351#issuecomment-222583109 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-6320][SQL] Move planLater method into G...

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

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

2016-05-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13392#issuecomment-222574008 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

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

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

  1   2   3   >