Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2700
@mohamagdy could you check - there are a bunch of errors
https://travis-ci.org/mohamagdy/zeppelin/builds/324495310
---
lix Cheung <felixcheun...@hotmail.com>
Closes #20340 from felixcheung/rstreamdoc.
(cherry picked from commit 2239d7a410e906ccd40aa8e84d637e9d06cd7b8a)
Signed-off-by: Felix Cheung <felixche...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wi
lix Cheung <felixcheun...@hotmail.com>
Closes #20340 from felixcheung/rstreamdoc.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/2239d7a4
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/2239d7a4
Diff: http:
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2700
looks like Jenkins has lost the test run result - could you kick that off
again? you can do that by closing and reopening this PR.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19528
Jenkins test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20255#discussion_r162261740
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1089,6 +1098,224 @@ streamingDf.join(staticDf, "type", "right_join&qu
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20272#discussion_r161979319
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -328,7 +328,7 @@ object SparkSubmit extends CommandLineUtils
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20267
I'm not 100% sure... I injected some error in jekyll but it stopped
immediately.
let me try to match the report condition more closely
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20232#discussion_r161683935
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/MultilayerPerceptronClassifier.scala
---
@@ -102,36 +102,6 @@ private[classification
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20232#discussion_r161686126
--- Diff: docs/sparkr.md ---
@@ -663,3 +663,4 @@ You can inspect the search path in R with
[`search()`](https://stat.ethz.ch/R-ma
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20204#discussion_r161677975
--- Diff: python/run-tests-with-coverage ---
@@ -0,0 +1,69 @@
+#!/usr/bin/env bash
+
+#
+# Licensed to the Apache Software Foundation
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20163#discussion_r161677327
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/EvaluatePython.scala
---
@@ -144,6 +145,7 @@ object EvaluatePython
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20267#discussion_r161444163
--- Diff: dev/create-release/release-build.sh ---
@@ -290,6 +290,8 @@ if [[ "$1" == "docs" ]]; then
cd docs
# TOD
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20267
@sameeragarwal FYI.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20267
[SPARK-23068][BUILD][RELEASE] doc build error from jekyll does not fail
## What changes were proposed in this pull request?
check exit code. note that not errors are reported via exit
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20264#discussion_r161406899
--- Diff: project/MimaBuild.scala ---
@@ -88,7 +88,7 @@ object MimaBuild {
def mimaSettings(sparkHome: File, projectRef: ProjectRef
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20263
thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20230
merged to master/2.3/2.2
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Repository: spark
Updated Branches:
refs/heads/branch-2.2 105ae8680 -> 7022ef800
[SPARK-23038][TEST] Update docker/spark-test (JDK/OS)
## What changes were proposed in this pull request?
This PR aims to update the followings in `docker/spark-test`.
- JDK7 -> JDK8
Spark 2.2+ supports JDK8
Repository: spark
Updated Branches:
refs/heads/branch-2.3 1f4a08b15 -> a335a49ce
[SPARK-23038][TEST] Update docker/spark-test (JDK/OS)
## What changes were proposed in this pull request?
This PR aims to update the followings in `docker/spark-test`.
- JDK7 -> JDK8
Spark 2.2+ supports JDK8
Repository: spark
Updated Branches:
refs/heads/master c3548d11c -> 7a3d0aad2
[SPARK-23038][TEST] Update docker/spark-test (JDK/OS)
## What changes were proposed in this pull request?
This PR aims to update the followings in `docker/spark-test`.
- JDK7 -> JDK8
Spark 2.2+ supports JDK8 only.
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20254
@henryr could you update this PR to only include EXCEPT DISTINCT without
the notes
---
-
To unsubscribe, e-mail: reviews
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20263
[SPARK-23069] fix R doc for describe missing text
## What changes were proposed in this pull request?
fix doc truncated
## How was this patch tested?
manually
You
Repository: spark
Updated Branches:
refs/heads/master afae8f2bc -> c3548d11c
[SPARK-23063][K8S] K8s changes for publishing scripts (and a couple of other
misses)
## What changes were proposed in this pull request?
Including the `-Pkubernetes` flag in a few places it was missed.
## How was
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20256
thanks! merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Repository: spark
Updated Branches:
refs/heads/branch-2.3 bcd87ae07 -> 1f4a08b15
[SPARK-23063][K8S] K8s changes for publishing scripts (and a couple of other
misses)
## What changes were proposed in this pull request?
Including the `-Pkubernetes` flag in a few places it was missed.
## How
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20254
I see, from reading that PR I think perhaps we should reference migration
guide in sql programming guide instead of putting the whole description here
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20080
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20230#discussion_r161367060
--- Diff: external/docker/spark-test/base/Dockerfile ---
@@ -15,14 +15,14 @@
# limitations under the License.
#
-FROM ubuntu:precise
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20255#discussion_r161366935
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1089,6 +1098,224 @@ streamingDf.join(staticDf, "type", "right_join&qu
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20255#discussion_r161366948
--- Diff: docs/structured-streaming-programming-guide.md ---
@@ -1089,6 +1098,224 @@ streamingDf.join(staticDf, "type", "right_join&qu
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20256#discussion_r161366655
--- Diff: dev/create-release/releaseutils.py ---
@@ -185,6 +185,7 @@ def get_commits(tag):
"graphx": "GraphX",
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20211#discussion_r161366590
--- Diff: python/pyspark/sql/group.py ---
@@ -233,6 +233,27 @@ def apply(self, udf):
| 2| 1.1094003924504583
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20256#discussion_r161366220
--- Diff: dev/create-release/releaseutils.py ---
@@ -185,6 +185,7 @@ def get_commits(tag):
"graphx": "GraphX",
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20232#discussion_r161365548
--- Diff: R/pkg/tests/fulltests/test_mllib_classification.R ---
@@ -382,10 +382,10 @@ test_that("spark.mlp", {
trainidxs <- base
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20254#discussion_r161365422
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1364,7 +1364,9 @@ def subtract(self, other):
""" Return a new :c
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20254#discussion_r161365371
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2873,6 +2873,7 @@ setMethod("intersect",
#' @rdname except
#' @export
#' @note except s
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20254#discussion_r161365416
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2873,6 +2873,7 @@ setMethod("intersect",
#' @rdname except
#' @export
#' @note except s
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20256#discussion_r161365293
--- Diff: dev/create-release/releaseutils.py ---
@@ -185,6 +185,7 @@ def get_commits(tag):
"graphx": "GraphX",
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20229
we need to get it to kick off R tests - could you touch one of the files
under R/?
also please update PR to include [SPARKR
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160874070
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/executor/Dockerfile ---
@@ -1,35 +0,0 @@
-#
-# Licensed to the Apache
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160874300
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
@@ -0,0 +1,97 @@
+#!/bin/bash
+#
+# Licensed
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20222
Wait
dev/release-tag.sh does this automatically though.
I just want to make sure we are not missing things (like R/DESCRIPTION)
I think maybe we should run a subset
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20151
+1... this is "undocumented" conf, sooo it's an expert one :)
---
-
To unsubscribe, e-mail: review
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20188
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Repository: spark
Updated Branches:
refs/heads/branch-2.3 ecc24ec7f -> 2db523959
[SPARK-22993][ML] Clarify HasCheckpointInterval param doc
## What changes were proposed in this pull request?
Add a note to the `HasCheckpointInterval` parameter doc that clarifies that
this setting is ignored
Repository: spark
Updated Branches:
refs/heads/master eaac60a1e -> 70bcc9d5a
[SPARK-22993][ML] Clarify HasCheckpointInterval param doc
## What changes were proposed in this pull request?
Add a note to the `HasCheckpointInterval` parameter doc that clarifies that
this setting is ignored when
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19290
argh, thanks for the reminder and the fix.
I knew calling internal method is going to bite us
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19290
Right we could bump the supported R version for the next release. It should
have minimal impact (since we are testing the close to the latest on
appveyor... somewhat internally)
lintr
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20188
Actually in R setCheckpointDir method is not attached to the SparkContext;
Iâd leave it as ânot setâ or ânot set in the sessionâ
https://spark.apache.org/docs/latest/api/R
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20193
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Repository: spark
Updated Branches:
refs/heads/master 0959aa581 -> 6a4206ff0
[SPARK-22998][K8S] Set missing value for SPARK_MOUNTED_CLASSPATH in the
executors
## What changes were proposed in this pull request?
The environment variable `SPARK_MOUNTED_CLASSPATH` is referenced in the
Repository: spark
Updated Branches:
refs/heads/branch-2.3 e79480e5d -> 47f975b42
[SPARK-22998][K8S] Set missing value for SPARK_MOUNTED_CLASSPATH in the
executors
## What changes were proposed in this pull request?
The environment variable `SPARK_MOUNTED_CLASSPATH` is referenced in the
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20158
let's get them fixed for 2.3?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20197
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Repository: spark
Updated Branches:
refs/heads/branch-2.3 911a4dbe7 -> a23c07ecb
[SPARK-21293][SPARKR][DOCS] structured streaming doc update
## What changes were proposed in this pull request?
doc update
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20197 from felixcheu
Repository: spark
Updated Branches:
refs/heads/master 8486ad419 -> 02214b094
[SPARK-21293][SPARKR][DOCS] structured streaming doc update
## What changes were proposed in this pull request?
doc update
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20197 from felixcheu
Repository: spark
Updated Branches:
refs/heads/branch-2.3 fd46a276c -> 911a4dbe7
[SPARK-21292][DOCS] refreshtable example
## What changes were proposed in this pull request?
doc update
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20198 from felixcheung/rrefreshdoc.
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20198
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Repository: spark
Updated Branches:
refs/heads/master f20131dd3 -> 8486ad419
[SPARK-21292][DOCS] refreshtable example
## What changes were proposed in this pull request?
doc update
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20198 from felixcheung/rrefreshdoc.
Proj
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20198
[SPARK-21292][DOCS] refreshtable example
## What changes were proposed in this pull request?
doc update
You can merge this pull request into a Git repository by running:
$ git
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20197
[SPARK-21293][SPARKR][DOCS] structured streaming doc update
## What changes were proposed in this pull request?
doc update
You can merge this pull request into a Git repository
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19290
Given that we are forking 2.3 and locking down the branch any time now, it
might make sense to stay with the "current version running on old centos
workers", even though th
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20193
yap I thought about it and it's fine.
could you review that PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160316799
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
@@ -0,0 +1,97 @@
+#!/bin/bash
+#
+# Licensed
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160316427
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/executor/Dockerfile ---
@@ -1,35 +0,0 @@
-#
-# Licensed to the Apache
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160316166
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
@@ -0,0 +1,97 @@
+#!/bin/bash
+#
+# Licensed
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160316590
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/Config.scala
---
@@ -29,17 +29,23 @@ private[spark] object
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20192#discussion_r160315955
--- Diff:
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile ---
@@ -41,7 +41,8 @@ COPY ${spark_jars} /opt/spark/jars
COPY
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19290
Shane is this going to affect one particular branch (eg. 2.3.0), or is it
going to be all branches and all test runs?
The changes are fairly substantial - if we need to back port
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20146
ok SGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20146
I think all dataset with a string order get indexed, as far as I recall?
Pick existing R dataset is just a convenience, we can also make up a few
lines of data if that works out better
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20164#discussion_r160020496
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala ---
@@ -170,21 +170,24 @@ final class OneVsRestModel private[ml
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20167
is putting secrets as plain text files a good practice..?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20154
that's good, I think we should still address the finer point of
https://github.com/apache/spark/pull/20154#pullrequestreview-86833216
- if docker hub can't build spark-base then pretty much
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2704
@prabhjyotsingh
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2701
I think test passed? it's too long and jenkins no longer has the build
@zjffdu
---
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2689
ping @sjoerdmulder
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20160
test passed, merged to master/2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Repository: spark
Updated Branches:
refs/heads/branch-2.3 5b524cc0c -> f9dcdbcef
[SPARK-22757][K8S] Enable spark.jars and spark.files in KUBERNETES mode
## What changes were proposed in this pull request?
We missed enabling `spark.files` and `spark.jars` in
Repository: spark
Updated Branches:
refs/heads/master cf0aa6557 -> 6cff7d19f
[SPARK-22757][K8S] Enable spark.jars and spark.files in KUBERNETES mode
## What changes were proposed in this pull request?
We missed enabling `spark.files` and `spark.jars` in
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20151#discussion_r159819670
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonWorkerFactory.scala ---
@@ -34,17 +34,25 @@ private[spark] class PythonWorkerFactory
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20078
hmm, I didn't know that was changed actually (SPARK-13723)
But it seems to me `spark.streaming.dynamicAllocation.minExecutors` is
still a valid approach. To match the non-streaming behavior
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20078
not saying about this change, but I've use streaming dynamic allocation
quite a bit back in the day.
but in this case I think simply is to set
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20146#discussion_r159584417
--- Diff: R/pkg/tests/fulltests/test_mllib_classification.R ---
@@ -348,12 +348,12 @@ test_that("spark.mlp", {
# Test r
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20129
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
nBy
## How was this patch tested?
manual, unit tests
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20129 from felixcheung/rwater.
(cherry picked from commit df95a908baf78800556636a76d58bba9b3dd943f)
Signed-off-by: Felix Cheung <felixche...@apache.org>
Project:
nBy
## How was this patch tested?
manual, unit tests
Author: Felix Cheung <felixcheun...@hotmail.com>
Closes #20129 from felixcheung/rwater.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/df95a908
Tree: http://git-wip-us.a
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16578
We are still merging changes to the 2.3 branch :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20078
hmm, that sounds like a different problem, why is numReceivers set to >
spark.cores.max?
---
-
To unsubscribe, e-m
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/18714
ah yes, please please :)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20106
Maybe. Other test files have that extra empty line though ;)
---
-
To unsubscribe, e-mail: reviews-unsubscr
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20129
[SPARK-22933][SPARKR] R Structured Streaming API for withWatermark,
trigger, partitionBy
## What changes were proposed in this pull request?
R Structured Streaming API
Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2695
what's the next step then?
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159106981
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -261,6 +261,17 @@ object SQLConf {
.booleanConf
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159106860
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -261,6 +261,17 @@ object SQLConf {
.booleanConf
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159106914
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -261,6 +261,17 @@ object SQLConf {
.booleanConf
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/19758
ping?
I'm mostly interested in SPARK-3162
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20118
@shivaram @HyukjinKwon what do you think about this approach?
---
-
To unsubscribe, e-mail: reviews-unsubscr
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/20118
[SPARK-22924][SPARKR] R API for sortWithinPartitions
## What changes were proposed in this pull request?
Add to `arrange` option to sort only within partition
## How
1101 - 1200 of 6371 matches
Mail list logo
