Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
Thanks for your help with this @mridulm @jerryshao .
Yes, sure, I am doing it, I'll ping you once they are ready, thanks.
---
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
This should also be backported to branch 2.2 and 2.3 @mridulm , this is a
regression.
@mgaido91 would you please create backport PRs for the separate branches?
---
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/21895
I merged to master, thanks for the work @mgaido91 !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94268/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94268 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94268/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94268 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94268/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94171/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94171 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94171/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94171 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94171/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94140/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #94140 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94140/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94140 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94140/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94127/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #94127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94127/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94111/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94111 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94111/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94127/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
sure @jerryshao , will do. Thanks for the review.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
Hi @mgaido91 would you please check it is auto-mergeable to branch 2.2/2.3,
if not please also repare the fix for the related branch once this is merged.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94111/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94084/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #94084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94084/testReport)**
for PR 21895 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
Ping @mridulm , would you please also take a review, thanks!
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #94084 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94084/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
retest this pelase
---
-
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/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93991/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93991 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93991/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93991 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93991/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
@mridulm would you please also take a review. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93894/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93894 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93894/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93894 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93894/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93881/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #93881 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93881/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21895
**[Test build #93881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93881/testReport)**
for PR 21895 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
thanks for comments @jerryshao . I will update this PR accordingly. Thanks.
---
-
To unsubscribe, e-mail:
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
I don't think the problem you mentioned is a big problem.
1. For the blacklist mechanism, we can have a time-based reviving mechanism
to check if permission is changed, compared to check
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
@jerryshao that can be done but I see mainly 2 problems IIUC:
- your suggestion about blacklisting has the same "caching" and "memory
leakage" problems of the solution proposed here, ie. if
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21895
My current thinking is to revert SPARK-20172 and improve the logging when
exception is met during the actual read.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93735/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #93735 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93735/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93735 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93735/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93726/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #93726 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93726/testReport)**
for PR 21895 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/21895
+CC @jerryshao
---
-
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/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
cc @mridulm too
---
-
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/21895
**[Test build #93726 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93726/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93719/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93719 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93719/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93719/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93676/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
**[Test build #93676 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93676/testReport)**
for PR 21895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21895
**[Test build #93676 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93676/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
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/21895
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/21895
**[Test build #93673 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93673/testReport)**
for PR 21895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93673/
Test FAILed.
---
1 - 100 of 105 matches
Mail list logo