Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
argh, sorry about the mistake, thank you for the fix @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
@cloud-fan oh good point, sorry that was an oversight on my part. Since it
was clean I just pushed it directly here:
https://github.com/apache/spark/commit/47a668c2f03d77078259531ddaccf80b001a8b5c
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22818
since this is a bug fix, shall we also backport it?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addi
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22818
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98520/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98520 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98520/testReport)**
for PR 22818 at commit
[`f42b1a8`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98520 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98520/testReport)**
for PR 22818 at commit
[`f42b1a8`](https://github.com/apache/spark/commit/f4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/4797/
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
thanks for the review @attilapiros, fixed the style issue, I think the
other one is OK as is.
---
-
To unsubscribe, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #4413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4413/testReport)**
for PR 22818 at commit
[`361bf02`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #4413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4413/testReport)**
for PR 22818 at commit
[`361bf02`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98504/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98504 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98504/testReport)**
for PR 22818 at commit
[`361bf02`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98504 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98504/testReport)**
for PR 22818 at commit
[`361bf02`](https://github.com/apache/spark/commit/36
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/4788/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98451/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98451/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/4755/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98451/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/ca
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22818
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98435/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98435/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98435/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/ca
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/4744/
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22818
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98371/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98371 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98371/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22818
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/4707/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22818
**[Test build #98371 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98371/testReport)**
for PR 22818 at commit
[`ca3efd8`](https://github.com/apache/spark/commit/ca
Github user squito commented on the issue:
https://github.com/apache/spark/pull/22818
I also changed the issue to SPARK-25904 -- there are some other things
related to encryption that it makes more sense to handle under SPARK-25827.
---
--
41 matches
Mail list logo