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

CLI: Finish check_if_any_deprecated_features_are_used implementation #12675

Merged

Conversation

dumbbell
Copy link
Member

@dumbbell dumbbell commented Nov 6, 2024

Why

The previous implementation bypassed the deprecated features subsystem. It only cared about classic mirrored queues and called some queue-related code directly to determine if this specific feature was used.

How

The command code is simplified by calling the deprecated subsystem to list used deprecated features instead.

References #12619.

@dumbbell dumbbell requested a review from Zerpet November 6, 2024 16:50
@dumbbell dumbbell self-assigned this Nov 6, 2024
@dumbbell dumbbell force-pushed the fix-check_if_any_deprecated_features_are_used-cli-command branch from 0297ec9 to 16ca325 Compare November 6, 2024 16:52
[Why]
The previous implementation bypassed the deprecated features subsystem.
It only cared about classic mirrored queues and called some
queue-related code directly to determine if this specific feature was
used.

[How]
The command code is simplified by calling the deprecated subsystem to
list used deprecated features instead.

References #12619.
@dumbbell dumbbell force-pushed the fix-check_if_any_deprecated_features_are_used-cli-command branch from 16ca325 to ddaea6f Compare November 11, 2024 09:13
@dumbbell dumbbell marked this pull request as ready for review November 11, 2024 10:30
@michaelklishin michaelklishin added this to the 4.1.0 milestone Nov 14, 2024
@michaelklishin michaelklishin merged commit d5063c7 into main Nov 14, 2024
271 checks passed
@michaelklishin michaelklishin deleted the fix-check_if_any_deprecated_features_are_used-cli-command branch November 14, 2024 18:57
dumbbell added a commit that referenced this pull request Nov 15, 2024
CLI: Finish `check_if_any_deprecated_features_are_used` implementation (backport #12675)
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