Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23124
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23124
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/23025#discussion_r236970732
--- Diff: R/pkg/R/DataFrame.R ---
@@ -767,6 +767,14 @@ setMethod("repartition",
#' using \code{spark.sql.shuffle.partitions}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23124
**[Test build #99357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99357/testReport)**
for PR 23124 at commit
Github user stczwd commented on the issue:
https://github.com/apache/spark/pull/22575
I hive send an email to Ryan Blue.
> > > Can you send a mail to Ryan blue for adding this SPIP topic in
tomorrow meeting. Meeting will be conducted tomorrow 05:00 pm PST. If you
confirm
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21957#discussion_r236965962
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -269,7 +269,8 @@ case class FileSourceScanExec(
}
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99348/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22163
**[Test build #99348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99348/testReport)**
for PR 22163 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23128
**[Test build #99355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99355/testReport)**
for PR 23128 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23153
**[Test build #99356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99356/testReport)**
for PR 23153 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23052
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23052
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23153
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23153
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23124#discussion_r236962499
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapBuilder.scala
---
@@ -0,0 +1,118 @@
+/*
+ * Licensed to
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/23128
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23052
**[Test build #99354 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99354/testReport)**
for PR 23052 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23052
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22590
**[Test build #99353 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99353/testReport)**
for PR 22590 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23124#discussion_r236955791
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapBuilder.scala
---
@@ -0,0 +1,118 @@
+/*
+ * Licensed to the
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23124#discussion_r236958252
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapBuilderSuite.scala
---
@@ -0,0 +1,91 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22590
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22590
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/22590
@HyukjinKwon I think it is not important. but our customers need this
feature.
Yeah, it is better to find a way to set the arbitrary parse settings options
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23086#discussion_r236957293
--- Diff: sql/core/src/main/java/org/apache/spark/sql/sources/v2/Table.java
---
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache Software
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/23128
python UT failed cause jvm crush.
retest this pleas.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/23124#discussion_r236952729
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/ArrayBasedMapBuilder.scala
---
@@ -0,0 +1,118 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23162
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23162
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99345/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23162
**[Test build #99352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99352/testReport)**
for PR 23162 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23160
**[Test build #99345 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99345/testReport)**
for PR 23160 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99346/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23160
**[Test build #99346 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99346/testReport)**
for PR 23160 at commit
GitHub user 10110346 opened a pull request:
https://github.com/apache/spark/pull/23162
[MINOR][DOC] Correct some document description errors
## What changes were proposed in this pull request?
Correct some document description errors.
## How was this patch tested?
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23124#discussion_r236949897
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetRowConverter.scala
---
@@ -558,8 +558,11 @@ private[parquet]
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99347/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23128
**[Test build #99347 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99347/testReport)**
for PR 23128 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/22575
> > Can you send a mail to Ryan blue for adding this SPIP topic in tomorrow
meeting. Meeting will be conducted tomorrow 05:00 pm PST. If you confirm then
we can also attend the meeting.
>
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23127
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23127
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user stczwd commented on the issue:
https://github.com/apache/spark/pull/22575
> Can you send a mail to Ryan blue for adding this SPIP topic in tomorrow
meeting. Meeting will be conducted tomorrow 05:00 pm PST. If you confirm then
we can also attend the meeting.
I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23083
**[Test build #99351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99351/testReport)**
for PR 23083 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23083
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23083
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23083
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99350/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23161
**[Test build #99350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99350/testReport)**
for PR 23161 at commit
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/22575
Can you send a mail to Ryan blue for adding this SPIP topic in tomorrow
meeting. Meeting will be conducted tomorrow 05:00 pm PST. If you confirm
then we can also attend the meeting.
Github user stczwd commented on the issue:
https://github.com/apache/spark/pull/22575
>
I have removed the 'stream' keyword.
> There is a DatasourceV2 community
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/22575
cc @koeninger
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/22575
There is a DatasourceV2 community synch meetup tomorrow which is
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23161
**[Test build #99350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99350/testReport)**
for PR 23161 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23161
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23144
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99349/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23144
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user huaxingao opened a pull request:
https://github.com/apache/spark/pull/23161
[SPARK-26189][R]Fix unionAll doc in SparkR
## What changes were proposed in this pull request?
Fix unionAll doc in SparkR
## How was this patch tested?
Manually ran
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23144
**[Test build #99349 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99349/testReport)**
for PR 23144 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23144
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/23144
Using an optional `normalize` function argument maybe OK, I will have a try.
---
-
To unsubscribe, e-mail:
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/23144
@srowen To adopt an optional `normalize` function argument, we may need to
create a new class `StringParam` and add the argument into it. But this will be
a breaking change, since existing
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23144
**[Test build #99349 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99349/testReport)**
for PR 23144 at commit
Github user guoxiaolongzte commented on a diff in the pull request:
https://github.com/apache/spark/pull/23104#discussion_r236929433
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -459,6 +459,7 @@ object LimitPushDown extends
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/22163
cc @kiszk @maropu
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22163
**[Test build #99348 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99348/testReport)**
for PR 22163 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23128
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user KyleLi1985 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23126#discussion_r236927771
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/linalg/distributed/RowMatrixSuite.scala
---
@@ -266,6 +266,16 @@ class RowMatrixSuite extends
Github user KyleLi1985 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23126#discussion_r236927721
--- Diff: mllib/src/test/java/org/apache/spark/ml/feature/JavaPCASuite.java
---
@@ -67,7 +66,7 @@ public void testPCA() {
JavaRDD dataRDD =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23128
**[Test build #99347 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99347/testReport)**
for PR 23128 at commit
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/23128#discussion_r236926403
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ShuffledRowRDD.scala ---
@@ -154,7 +156,10 @@ class ShuffledRowRDD(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23055#discussion_r236926113
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -74,8 +74,13 @@ private[spark] abstract class BasePythonRunner[IN,
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/23154
LGTM,thanks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23159
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99342/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23159
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23159
**[Test build #99342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99342/testReport)**
for PR 23159 at commit
Github user lcqzte10192193 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23154#discussion_r236923346
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java
---
@@ -510,42 +510,42 @@ public
Github user lcqzte10192193 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23154#discussion_r236923308
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -204,10 +204,10 @@ case class
Github user heary-cao commented on a diff in the pull request:
https://github.com/apache/spark/pull/23151#discussion_r236922709
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -105,5 +105,16 @@ abstract class SparkFunSuite
logInfo(s"\n\n=
Github user stczwd commented on the issue:
https://github.com/apache/spark/pull/22575
@sujithjay
Please refer
[SPARK-24630](https://issues.apache.org/jira/browse/SPARK-24630) for more
details.
---
-
To
Github user 10110346 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23154#discussion_r236920634
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedRleValuesReader.java
---
@@ -510,42 +510,42 @@ public void
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23154#discussion_r236919935
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala
---
@@ -258,7 +258,7 @@ case class GeneratorOuter(child:
Github user heary-cao commented on a diff in the pull request:
https://github.com/apache/spark/pull/23151#discussion_r236919725
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -105,5 +105,16 @@ abstract class SparkFunSuite
logInfo(s"\n\n=
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/23154#discussion_r236919395
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -204,10 +204,10 @@ case class UnresolvedGenerator(name:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/23154
@lcqzte10192193 I am sorry for my misunderstanding The original code in
`VectorizedRleValuesReader.java` was correct. Could you please revert you
change?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23160
**[Test build #99346 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99346/testReport)**
for PR 23160 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23160
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23160
**[Test build #99345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99345/testReport)**
for PR 23160 at commit
1 - 100 of 491 matches
Mail list logo
