Skip to content

[Studio] Surface consumer group Admin failures - #966

Closed
Aias00 wants to merge 1 commit into
apache:rocketmq-studiofrom
Aias00:fix/studio-consumer-group-admin-errors-965
Closed

[Studio] Surface consumer group Admin failures#966
Aias00 wants to merge 1 commit into
apache:rocketmq-studiofrom
Aias00:fix/studio-consumer-group-admin-errors-965

Conversation

@Aias00

@Aias00 Aias00 commented Aug 4, 2026

Copy link
Copy Markdown

Closes #965

Summary

  • preserve the successful offline fallback only for MQClientException with CONSUMER_NOT_ONLINE
  • convert timeout, broker, ACL, and other Admin call failures into explicit HTTP 502 business responses
  • add regression coverage for offline, timeout, and broker/ACL failure paths

Related

This is intentionally separate from #786 / #787, which cover unconfigured default providers rather than the real RocketMQAdminClientImpl Admin path.

Verification

JAVA_HOME=$(/usr/libexec/java_home -v 21) PATH="$JAVA_HOME/bin:$PATH" mvn -Dtest=RocketMQAdminClientImplTest test

Copilot AI lite review requested due to automatic review settings August 4, 2026 09:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Aias00

Aias00 commented Aug 5, 2026

Copy link
Copy Markdown
Author

Superseded by #1025. The replacement is rebased on rocketmq-studio and contains only the #965 implementation and focused regression tests; closing this PR to avoid retaining unrelated historical changes.

@Aias00

Aias00 commented Aug 5, 2026

Copy link
Copy Markdown
Author

Closed in favor of the clean replacement #1025.

@Aias00 Aias00 closed this Aug 5, 2026
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.

2 participants