sunnysabor commented on PR #7415: URL: https://github.com/apache/shenyu/pull/7415#issuecomment-5945383471
All requested changes are now on the current head `72dae9befc65b5683805a309631f01fbe988fe1d`: the canonical plugin alias, per-run URI correlation, RabbitMQ connect/handshake timeouts, and corrected case name. The follow-up HTTP long-polling failure was also fixed by handling an empty selector handle and covered by a unit regression test. Validation is now complete on this head: the focused RabbitMQ plugin test passes (8 tests), `pr_build` and the normal build pass, and the aggregate E2E workflow passes, including `e2e-logging-rabbitmq-compose` across WebSocket, HTTP long-polling, and Zookeeper sync, plus the Apache Dubbo case. License/header, Java analysis, CodeQL, Docker image build, integration, K8s, and other E2E jobs are green as well. Could you please re-review the updated head when convenient? Thanks for the detailed feedback. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
