Skip to content

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Mar 7, 2025

Which issue does this PR close?

Rationale for this change

there is a vulnerability in old versions of ring that is triggering a CI failure

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@alamb alamb marked this pull request as ready for review March 7, 2025 12:42
Copy link
Contributor

@zhuqi-lucas zhuqi-lucas left a comment

Choose a reason for hiding this comment

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

LGTM Thank you @alamb !

Copy link
Member

@jonahgao jonahgao left a comment

Choose a reason for hiding this comment

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

@jonahgao jonahgao merged commit 755f9a5 into apache:main Mar 7, 2025
26 checks passed
@alamb alamb deleted the alamb/update_ring branch March 7, 2025 14:24
danila-b pushed a commit to danila-b/datafusion that referenced this pull request Mar 8, 2025
alamb added a commit to alamb/datafusion that referenced this pull request Mar 14, 2025
@alamb
Copy link
Contributor Author

alamb commented Mar 14, 2025

xudong963 pushed a commit that referenced this pull request Mar 14, 2025
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.

security_audit CI check is failing

3 participants