Skip to content

Change native library default to epoll over io_uring #3278

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

Merged
merged 2 commits into from
May 23, 2025

Conversation

thachlp
Copy link
Contributor

@thachlp thachlp commented Apr 26, 2025

Closes #3276

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

Copy link
Contributor Author

@thachlp thachlp left a comment

Choose a reason for hiding this comment

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

@tishun Please help review it 🙇

@tishun
Copy link
Collaborator

tishun commented May 9, 2025

We would need to address the two issues - #3275, #3276 and #3277 - separately:

The first and second PR is targeted for now, the last one will be merged when we are ready to start for on 7.x
#3275 and #3276 can be addressed with the same PR (this one)

Does that make sense?

@thachlp thachlp requested a review from tishun May 19, 2025 09:15
@tishun tishun merged commit 5434d0c into redis:main May 23, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display warning message in log when a native library conflict arises
2 participants