Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-39863][BUILD][FOLLOWUP] Update dependency manifest for Hadoop 3 #37447

Closed
wants to merge 1 commit into from
Closed

[SPARK-39863][BUILD][FOLLOWUP] Update dependency manifest for Hadoop 3 #37447

wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Aug 9, 2022

What changes were proposed in this pull request?

This PR aims to update the dependency manifest for Hadoop 3.

HADOOP-18344 changes AWS SDK at Apache Hadoop 3.3.4 RC1.

Why are the changes needed?

#37281 missed this inconsistency.

Does this PR introduce any user-facing change?

No. This will recover the dependency check CI job.

How was this patch tested?

Pass the CI on this job.

@github-actions github-actions bot added the BUILD label Aug 9, 2022
@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Aug 9, 2022

cc @sunchao
Also, cc @viirya

@viirya
Copy link
Member

viirya commented Aug 9, 2022

I think it is just forgot to run manifest update? As pom is not changed here.

@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Aug 9, 2022

Yes, right. This change was introduced at Apache Hadoop 3.3.4 RC1. So, RC0-based generated manifest file missed that.

@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Aug 9, 2022

Thank you, @viirya .
Since this is only one of Linter, I'll merge this to recover master branch and PR builders. I manually verified this patch.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-39863 branch August 9, 2022 07:09
@viirya
Copy link
Member

viirya commented Aug 9, 2022

Thanks @dongjoon-hyun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants