Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20611
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/22110#discussion_r210212586
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/AbstractDataType.scala
---
@@ -115,6 +115,8 @@ protected[sql] abstract class AtomicType
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
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/22111
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/22111
**[Test build #94797 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94797/testReport)**
for PR 22111 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21889
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94790/
Test FAILed.
---
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21909
@HyukjinKwon @maropu Please, have a look at the PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22013#discussion_r210193591
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/higherOrderFunctions.scala
---
@@ -497,6 +497,65 @@ case class
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21561#discussion_r210205064
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/BisectingKMeans.scala ---
@@ -151,13 +152,10 @@ class BisectingKMeans private (
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22107
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94791/
Test PASSed.
---
Github user eyalfa commented on the issue:
https://github.com/apache/spark/pull/17400
thanks @maropu I appreciate this.
must say I'm pretty surprised a bug like that lives so long...
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
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/22111
**[Test build #94799 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94799/testReport)**
for PR 22111 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/22104
Thanks @HyukjinKwon and @cloud-fan ! I will take a look
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21718
@bjkonglu @bethunebtj @wguangliang
Update: I thought about splitting execution tasks and data partitions
(`spark.sql.shuffle.partitions`), and turned out it can be achieved by calling
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22111
cc @cloud-fan @kiszk @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21123
BTW, do we consider Datasource V1 fallback too?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94797/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22111
**[Test build #94797 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94797/testReport)**
for PR 22111 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94793/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21661
**[Test build #94792 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94792/testReport)**
for PR 21661 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21661
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/22110
**[Test build #94795 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94795/testReport)**
for PR 22110 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22107
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/22107
**[Test build #94791 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94791/testReport)**
for PR 22107 at commit
GitHub user mn-mikke opened a pull request:
https://github.com/apache/spark/pull/22110
[SPARK-25122][SQL] Deduplication of supports equals code
## What changes were proposed in this pull request?
The method ```*supportEquals``` determining whether elements of a data type
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94798 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94798/testReport)**
for PR 22009 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
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/22009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22110#discussion_r210215352
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/AbstractDataType.scala
---
@@ -115,6 +115,8 @@ protected[sql] abstract class AtomicType
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94793 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94793/testReport)**
for PR 20611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22110
**[Test build #94795 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94795/testReport)**
for PR 22110 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94796 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94796/testReport)**
for PR 20611 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21661
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94792/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94798 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94798/testReport)**
for PR 22009 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21889
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/21889
**[Test build #94790 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94790/testReport)**
for PR 21889 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21860
**[Test build #94794 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94794/testReport)**
for PR 21860 at commit
Github user mn-mikke commented on the issue:
https://github.com/apache/spark/pull/22110
cc @ueshin @cloud-fan
---
-
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/20611
Can one of the admins verify this patch?
---
-
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/22110#discussion_r210207729
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/AbstractDataType.scala
---
@@ -115,6 +115,8 @@ protected[sql] abstract class AtomicType
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/22111
[SPARK-25123][SQL] Use Block to track code in SimpleExprValue
## What changes were proposed in this pull request?
`SimpleExprValue` carries some java code which is a rvalue. Now the code
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21860
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/21860
**[Test build #94794 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94794/testReport)**
for PR 21860 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21860
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94794/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94798/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
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/22110
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22110
Can one of the admins verify this patch?
---
-
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/22110#discussion_r210213125
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/AbstractDataType.scala
---
@@ -115,6 +115,8 @@ protected[sql] abstract class AtomicType
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22110
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/22110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94795/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #94796 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94796/testReport)**
for PR 20611 at commit
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r210271735
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -91,6 +91,13 @@ private[spark] class Client(
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r210274748
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/BatchEvalPythonExec.scala
---
@@ -69,7 +67,7 @@ case class
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22093
`Should I delete current UT from FileSuit?`
I think current UT in `FileSuite` is unnecessarily, you can leave it and
wait for other reviewer's opinion.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94799/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22111
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22081
making a test-time option is a reasonable idea -getting the unlimited JCE
on the test machines (they don't right now) would remove the need for this
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21889
**[Test build #94805 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94805/testReport)**
for PR 21889 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
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/22009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22007
The bump is fine but this is not for making it congruent with Stocator,
which is just an external connector.
---
-
To
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r210317086
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/BatchEvalPythonExec.scala
---
@@ -69,7 +67,7 @@ case class
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21977
@holdenk, what could be the cause?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22099
To be clear you think this passed because it still uses jets3t and that
still brings in BC? Then we can maybe merge this and rebase the other change to
find out. This update won't have changed that
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
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/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94796/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22001
**[Test build #94801 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94801/testReport)**
for PR 22001 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22009
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/22101
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/21977
**[Test build #94804 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94804/testReport)**
for PR 21977 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21977
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/21977
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/21146
Closing now we have a test run with the combination of: No jets3t, no
bouncy castle, upgraded kinesis. *all the kinesis tests now run*
---
Github user steveloughran closed the pull request at:
https://github.com/apache/spark/pull/21146
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21835#discussion_r210326849
--- Diff: R/pkg/R/functions.R ---
@@ -3320,7 +3321,7 @@ setMethod("explode",
#' @aliases sequence sequence,Column-method
#' @note sequence since
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22110
**[Test build #94800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94800/testReport)**
for PR 22110 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22001
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/22001
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/22111
**[Test build #94799 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94799/testReport)**
for PR 22111 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20637
**[Test build #94806 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94806/testReport)**
for PR 20637 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21123
v1 fallback should be used as a temporary workaround, not a long-term
solution to fix the gap between v1 and v2.
---
-
To
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21123
Yea I agree. I was just wondering if we plan similar stuff for all
datasource v1 not only FileFormat one since it looks the reason described above
potentially apply to all datasource v1 I
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22110#discussion_r210282476
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/AbstractDataType.scala
---
@@ -115,6 +115,8 @@ protected[sql] abstract class AtomicType
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22101
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/22101
**[Test build #94803 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94803/testReport)**
for PR 22101 at commit
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/22099
> To be clear you think this passed because it still uses jets3t and that
still brings in BC?
correct
> Then we can maybe merge this and rebase the other change to find out.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #94802 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94802/testReport)**
for PR 22009 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21977
Actually looking at the Jenkins failures, could be the cause of some of
them as well (certainly the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
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/20637
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/21308
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/22106
**[Test build #4261 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4261/testReport)**
for PR 22106 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22106
**[Test build #4260 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4260/testReport)**
for PR 22106 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/22106#discussion_r210387003
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaContinuousSinkSuite.scala
---
@@ -40,12 +40,7 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22106
**[Test build #4262 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4262/testReport)**
for PR 22106 at commit
1 - 100 of 510 matches
Mail list logo