Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21707
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21761
Please do not merge it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21442
@HyukjinKwon The code has a bug. Please read the discussion before you
trigger the retest.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21770
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21780
Yup, please do.
---
-
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/21770
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93042/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21711
**[Test build #93082 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93082/testReport)**
for PR 21711 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21442
Also, see the stale PRs and see what we have delayed to take a look. Please
consider partly this is what we should have checked and reviewed earlier,
and/or partly authors haven't updated their
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19538
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93054/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19538
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21442
@HyukjinKwon Thank you for trying to trigger the tests but it will not help
if you got many pings within one hour. To save the resource, we need to do more
investigation before blindly
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21442
If you want to help this, we should ping the reviewers/committers based on
the priority of these PRs. Also, we should not trigger the pings within a short
period. Please reduce the size of the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21776
**[Test build #93036 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93036/testReport)**
for PR 21776 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21184
**[Test build #93037 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93037/testReport)**
for PR 21184 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21562
sure, LGTM.
---
-
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/21780
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/21780
**[Test build #93075 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93075/testReport)**
for PR 21780 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21780
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-unified/987/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19449
Hm, shall we leave this closed then?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21657
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20856
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21240
ping @dilipbiswal for an update.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21711#discussion_r202568645
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -138,17 +138,36 @@ private[spark] class
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21707
oops, branch-2.2 doesn't have the fix yet
`https://github.com/apache/spark/pull/21779`.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20405
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/20405
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93078/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20405
**[Test build #93078 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93078/testReport)**
for PR 20405 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21102
cc @ueshin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21779
Merged into branch-2.2.
---
-
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/21779
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/21352
**[Test build #93062 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93062/testReport)**
for PR 21352 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20864
@HyukjinKwon We also do not need to trigger the test for this PR. This fix
does not look good based on the above comment from @liufengdb
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21442
@HyukjinKwon All the involved reviewers will get a ping. This is annoying
to see many pings within one hour, right? My suggestion is to read the comments
before triggering the test
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21770
**[Test build #93045 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93045/testReport)**
for PR 21770 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20856
Sure. Let me give a shot to reproduce it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21582
@dongjoon-hyun Could you give how the benchmark works? What is the workload
pattern? How does the benchmark invoke Spark?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21352
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/21352
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93062/
Test FAILed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21442
@gatorsmile, can I just see if it the build passes against the latest build?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21770
**[Test build #93042 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93042/testReport)**
for PR 21770 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20864
@gatorsmile, I triggered tests for PRs where there's committer's command
for it. I don't know why or who started to block the tests in Jenkins for what
reason. If the author is willing to
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21442
This should get updated, right? I should give a ping here anyway. I wonder
why triggering retesting so matters for some PRs.
Probably, you are mostly talking about "ok to test" that I
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19538
**[Test build #93054 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93054/consoleFull)**
for PR 19538 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21711
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/21770
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93045/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21770
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/21711
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-unified/994/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21711
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21442
I am not blindly triggering the test. I skimmed and only re-triggered tests
some PRs while I am skimming stale PRs.
For "ok to test", I haven't also blindly re-triggered. I did where
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20405
@HyukjinKwon Does the existing AnalysisBarrier introduce a regression when
users use the hint like `df1.hint("broadcast", "t")`?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21755
**[Test build #4214 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4214/testReport)**
for PR 21755 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21589
**[Test build #93031 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93031/testReport)**
for PR 21589 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21589
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/21773
**[Test build #93035 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93035/testReport)**
for PR 21773 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21589
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93031/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21776
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/21776
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-unified/974/
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21755
I'm sure the failure is spurious, so merged to master.
PS see https://spark.apache.org/contributing.html
---
-
To
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21764#discussion_r202556147
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -127,6 +127,14 @@ object SQLConf {
}
}
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/21780
[SPARK-23259][SQL] Clean up legacy code around hive external catalog and
HiveClientImpl
## What changes were proposed in this pull request?
Three legacy statements are removed by this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21624
**[Test build #93064 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93064/testReport)**
for PR 21624 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21624
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19001
Hi all, any updates on this PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21657
Thanks! Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17092
ping @Yunni
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20405
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20095
ping @MrBago
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21079
ping @10110346 for ^
---
-
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/21442
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/21442
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-unified/990/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17086
**[Test build #93068 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93068/testReport)**
for PR 17086 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20864
**[Test build #93061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93061/testReport)**
for PR 20864 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93069/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21525
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
401 - 474 of 474 matches
Mail list logo