[GitHub] spark issue #21255: [SPARK-24186][R][SQL]change reverse and concat to collec...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21255 @huaxingao, that's absolutely fine. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21291 **[Test build #90496 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90496/testReport)** for PR 21291 at commit

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21230 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90488/ Test FAILed. ---

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21230 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21230 **[Test build #90488 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90488/testReport)** for PR 21230 at commit

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21291 Changed `outputPartitioning` changes executed plans. E.g. in `WholeStageCodegenSuite`, a query like `spark.range(3).groupBy("id").count().orderBy("id")`. Its executed plan changes from

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3128/

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21291 **[Test build #90495 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90495/testReport)** for PR 21291 at commit

[GitHub] spark issue #21255: [SPARK-24186][R][SQL]change reverse and concat to collec...

Github user huaxingao commented on the issue: https://github.com/apache/spark/pull/21255 @felixcheung @HyukjinKwon @viirya I am thinking of closing this one and open a new PR if no objection. It's messy to resolve the conflicts because I have quite a few patches. Sorry for my

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21291 retest this please. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark pull request #21106: [SPARK-23711][SQL][WIP] Add fallback logic for Un...

Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/21106#discussion_r187520079 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodegenObjectFactory.scala --- @@ -0,0 +1,80 @@ +/* + * Licensed

[GitHub] spark pull request #21106: [SPARK-23711][SQL][WIP] Add fallback logic for Un...

Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/21106#discussion_r187519505 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/InterpretedUnsafeProjection.scala --- @@ -87,12 +87,11 @@ class

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90487/ Test FAILed. ---

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21291 **[Test build #90487 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90487/testReport)** for PR 21291 at commit

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21118 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3127/

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21118 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21118 **[Test build #90494 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90494/testReport)** for PR 21118 at commit

[GitHub] spark issue #21255: [SPARK-24186][R][SQL]change reverse and concat to collec...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21255 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3126/

[GitHub] spark issue #21255: [SPARK-24186][R][SQL]change reverse and concat to collec...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21255 Build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark pull request #21276: [SPARK-24216][SQL] Spark TypedAggregateExpression...

Github user fangshil commented on a diff in the pull request: https://github.com/apache/spark/pull/21276#discussion_r187518454 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -2715,6 +2715,66 @@ private[spark] object Utils extends Logging {

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/21118 Retest this please. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21255: [SPARK-24186][R][SQL]change reverse and concat to collec...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21255 **[Test build #90493 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90493/testReport)** for PR 21255 at commit

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3125/

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90490/ Test FAILed. ---

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21106 **[Test build #90490 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90490/testReport)** for PR 21106 at commit

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21129 **[Test build #90492 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90492/testReport)** for PR 21129 at commit

[GitHub] spark issue #16578: [SPARK-4502][SQL] Parquet nested column pruning

Github user mallman commented on the issue: https://github.com/apache/spark/pull/16578 BTW I’ve been and am currently traveling with a busy itinerary. I haven’t started work on this and probably won’t get to work on it until Monday at the very earliest. > On May 5,

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90491/ Test FAILed. ---

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21129 **[Test build #90491 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90491/testReport)** for PR 21129 at commit

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3124/

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21129 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #17086: [SPARK-24101][ML][MLLIB] ML Evaluators should use weight...

Github user WeichenXu123 commented on the issue: https://github.com/apache/spark/pull/17086 LGTM. @jkbradley @mengxr Would you mind take a look ? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

[GitHub] spark issue #21129: [SPARK-7132][ML] Add fit with validation set to spark.ml...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21129 **[Test build #90491 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90491/testReport)** for PR 21129 at commit

[GitHub] spark pull request #21255: [SPARK-24186][R][SQL]change reverse and concat to...

Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/21255#discussion_r187515108 --- Diff: R/pkg/R/functions.R --- @@ -1256,7 +1259,7 @@ setMethod("quarter", #' @details #' \code{reverse}: Reverses the string column and returns

[GitHub] spark pull request #21255: [SPARK-24186][R][SQL]change reverse and concat to...

Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/21255#discussion_r187515152 --- Diff: R/pkg/R/functions.R --- @@ -2047,18 +2050,8 @@ setMethod("countDistinct", #' \code{concat}: Concatenates multiple input columns together into

[GitHub] spark pull request #21278: [SPARKR] Require Java 8 for SparkR

Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/21278#discussion_r187514621 --- Diff: R/pkg/R/client.R --- @@ -60,13 +60,48 @@ generateSparkSubmitArgs <- function(args, sparkHome, jars, sparkSubmitOpts, pack combinedArgs

[GitHub] spark pull request #21298: [SPARK-24198][SparkR][SQL] Adding slice function ...

Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/21298#discussion_r187514450 --- Diff: R/pkg/R/functions.R --- @@ -3124,6 +3125,23 @@ setMethod("size", column(jc) }) +#' @details +#'

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90485/ Test PASSed. ---

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21106 **[Test build #90485 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90485/testReport)** for PR 21106 at commit

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21163 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90489/ Test PASSed. ---

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21163 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21163 **[Test build #90489 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90489/testReport)** for PR 21163 at commit

[GitHub] spark issue #21290: [SPARK-24241][Submit]Do not fail fast when dynamic resou...

Github user yaooqinn commented on the issue: https://github.com/apache/spark/pull/21290 cc @srowen The last code change seems tp be related to you, plz help to review, thanks --- - To unsubscribe, e-mail:

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3123/

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21106 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21106 Any more comments on this? @hvanhovell @cloud-fan --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21106 **[Test build #90490 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90490/testReport)** for PR 21106 at commit

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187507940 --- Diff: core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala --- @@ -93,6 +94,10 @@ private[spark] class EventLoggingListener(

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187507139 --- Diff: core/src/main/scala/org/apache/spark/scheduler/PeakExecutorMetrics.scala --- @@ -0,0 +1,127 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187506958 --- Diff: core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala --- @@ -169,6 +179,27 @@ private[spark] class EventLoggingListener(

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187506780 --- Diff: core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala --- @@ -169,6 +179,27 @@ private[spark] class EventLoggingListener(

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21163 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21163 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3122/

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21230 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21230 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3121/

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21163 **[Test build #90489 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90489/testReport)** for PR 21163 at commit

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3120/

[GitHub] spark issue #21163: [SPARK-24097][ML] Instrumentation improvements - RandomF...

Github user WeichenXu123 commented on the issue: https://github.com/apache/spark/pull/21163 Jenkins, test this please. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21291 **[Test build #90487 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90487/testReport)** for PR 21291 at commit

[GitHub] spark issue #21230: [SPARK-24172][SQL] we should not apply operator pushdown...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21230 **[Test build #90488 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90488/testReport)** for PR 21230 at commit

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21291 retest this please. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21106 I agree. It is what I saw when I've tried to change it locally. --- - To unsubscribe, e-mail:

[GitHub] spark issue #21106: [SPARK-23711][SQL][WIP] Add fallback logic for UnsafePro...

Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/21106 I took a deeper look, changing to case class needs a lot of changes, let's follow your previous approach. --- - To

[GitHub] spark pull request #21291: [SPARK-24242][SQL] RangeExec should have correct ...

Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/21291#discussion_r187502412 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala --- @@ -345,6 +345,16 @@ case class RangeExec(range:

[GitHub] spark issue #19183: [SPARK-21960][Streaming] Spark Streaming Dynamic Allocat...

Github user karth295 commented on the issue: https://github.com/apache/spark/pull/19183 @sansagara go for it -- it'll be a few days until I'll have time to look at this again. I'll close my PR if/when you make a new one :) ---

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21291 retest this please. On Fri, May 11, 2018, 10:33 AM UCB AMPLab wrote: > Test FAILed. > Refer to this link for build results (access rights to CI server

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21291 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90484/ Test FAILed. ---

[GitHub] spark issue #21291: [SPARK-24242][SQL] RangeExec should have correct outputO...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21291 **[Test build #90484 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90484/testReport)** for PR 21291 at commit

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21153 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90486/ Test PASSed. ---

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21153 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21153 **[Test build #90486 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90486/testReport)** for PR 21153 at commit

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187501194 --- Diff: core/src/main/scala/org/apache/spark/Heartbeater.scala --- @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] spark pull request #21221: [SPARK-23429][CORE] Add executor memory metrics t...

Github user edwinalu commented on a diff in the pull request: https://github.com/apache/spark/pull/21221#discussion_r187501157 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -1753,9 +1766,21 @@ class DAGScheduler(

[GitHub] spark pull request #21296: [SPARK-24244][SQL] CSV column pruning

Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/21296#discussion_r187500240 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala --- @@ -267,7 +267,7 @@ class CSVSuite extends

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21153 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21153 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3119/

[GitHub] spark pull request #21296: [SPARK-24244][SQL] CSV column pruning

Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/21296#discussion_r187499921 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala --- @@ -73,11 +64,24 @@ class UnivocityParser(

[GitHub] spark issue #21296: [SPARK-24244][SQL] CSV column pruning

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21296 mind fixing the PR title? sounds we have never implemented the "column pruning" before in CSV. --- - To unsubscribe,

[GitHub] spark pull request #21296: [SPARK-24244][SQL] CSV column pruning

Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/21296#discussion_r187499797 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVBenchmarks.scala --- @@ -0,0 +1,92 @@ +/* + * Licensed to

[GitHub] spark pull request #21294: [SPARK-24197][SparkR][SQL] Adding array_sort func...

Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/21294 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #21298: [SPARK-24198][SparkR][SQL] Adding slice function to Spar...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21298 @mn-mikke, let's do this with few more functions together next time per the suggestions from @felixcheung and @shivaram who're PMCs used to R side better than me. ---

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21153 **[Test build #90486 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90486/testReport)** for PR 21153 at commit

[GitHub] spark issue #21153: [SPARK-24058][ML][PySpark] Default Params in ML should b...

Github user viirya commented on the issue: https://github.com/apache/spark/pull/21153 Done. Can you try again? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark pull request #21190: [SPARK-22938][SQL][followup] Assert that SQLConf....

Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/21190 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark issue #21294: [SPARK-24197][SparkR][SQL] Adding array_sort function to...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21294 Merged to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21297: [SPARK-24246][SQL]Improve AnalysisException by setting t...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21297 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21297: [SPARK-24246][SQL]Improve AnalysisException by setting t...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21297 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90479/ Test PASSed. ---

[GitHub] spark issue #21297: [SPARK-24246][SQL]Improve AnalysisException by setting t...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21297 **[Test build #90479 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90479/testReport)** for PR 21297 at commit

[GitHub] spark issue #21190: [SPARK-22938][SQL][followup] Assert that SQLConf.get is ...

Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21190 Merged to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user jose-torres commented on the issue: https://github.com/apache/spark/pull/21118 LGTM, although as you mentioned I think it'd definitely be valuable to follow up and understand why some operators insist on UnsafeRow even though this isn't what SparkPlan declares as the row

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21118 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21118: SPARK-23325: Use InternalRow when reading with DataSourc...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21118 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90481/ Test FAILed. ---

  1   2   3   4   5   >