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

HDDS-12152. Stop testing with Hadoop 3.1.2 #7773

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Remove Hadoop 3.1.2 from acceptance test matrix. Only test with 2.10.2 and 3.4.1.

https://issues.apache.org/jira/browse/HDDS-12152

How was this patch tested?

CI:
https://github.com/adoroszlai/ozone/actions/runs/13025669367/job/36334632022

@adoroszlai adoroszlai self-assigned this Jan 29, 2025
@jojochuang
Copy link
Contributor

Once 3.1.2 is removed from support matrix, we can remove LeaseRecovery and SafeMode clalsses from source code. They were introduced as a hack to maintain backward compat with 3.1/3.2.

* FIXME: Hack: This is copied from Hadoop 3.3.6. Remove this interface once

Can be a separate task.

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a comment to #5121
last time we discussed it we wanted to keep Hadoop 3.1

@adoroszlai
Copy link
Contributor Author

/pending response from discussion

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marking this issue as un-mergeable as requested.

Please use /ready comment when it's resolved.

Please note that the PR will be closed after 21 days of inactivity from now. (But can be re-opened anytime later...)

response from discussion

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

Successfully merging this pull request may close these issues.

2 participants