dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1238778068
Thank you for your review, comments, and help, @srowen , @Yikun , @sunchao ,
@steveloughran . It seems that there is no other concerns, I'll merge this.
--
This is an automated
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1235772123
There is no 3.0.0 yet. :)
--
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234809633
This PR passed CI here.
-
https://github.com/dongjoon-hyun/spark/runs/8139684479?check_suite_focus=true
--
This is an automated message from the Apache Git Service.
To respond
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234807199
Ur, wait. @steveloughran . It's Java 8, isn't it?
https://github.com/GoogleCloudDataproc/hadoop-connectors/blob/8453ce7ce7510e983bae7470909fbd02704c0539/pom.xml#L76-L77
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234803915
Thank you for review, @steveloughran .
> note that the gcs connector (at leasts the builds off their master) are
java 11 only; not sure where that stands w.r.t older releases
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234507890
Thank you so much, @Yikun . Now, it seems to work on my three PRs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234388728
1. I checked that mine is the same with you.
https://user-images.githubusercontent.com/9700541/187943970-bd5d40bf-8545-4d50-b7eb-16fc4a0440d8.png;>
2. Let me try to clean
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234077478
It's weird. IIRC, I didn't change anything from my previous repo either when
your PR applied this change.
--
This is an automated message from the Apache Git Service.
To respond
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234071272
I'm already allowing all of them.
https://user-images.githubusercontent.com/9700541/187891526-5938feb5-d380-4574-a81a-9b621779dead.png;>
--
This is an automated message
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234055469
Thank you, but `Base Image Build` phase failed three times already .
https://user-images.githubusercontent.com/9700541/187888992-48c0292b-2586-421b-8f9e-9b514ab35cb2.png;>
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1234014605
The failure is irrelevant to this PR. It seems that the base image
publishing is broken again. cc @Yikun
```
#33 ERROR: failed commit on ref
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1233372943
- Yes, only for better GCS support for the users who use `-Phadoop-cloud`.
- Apache Spark distribution doesn't use `-Phadoop-cloud` during our release
process. So, the publish
dongjoon-hyun commented on PR #37745:
URL: https://github.com/apache/spark/pull/37745#issuecomment-1233311486
cc @sunchao , @steveloughran
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
13 matches
Mail list logo
