sunnysabor commented on PR #7415: URL: https://github.com/apache/shenyu/pull/7415#issuecomment-5945228583
CI found a real integration gap on the HTTP long-polling run: RabbitMQ E2E creates a selector without a per-selector override (using the configured global RabbitMQ client), but `LoggingRabbitmqPluginDataHandler.handlerSelector()` dereferenced the null map returned for an empty handle. That left the log unconsumed and caused `testRabbitMqLog` to time out. Fixed this on current head in commit `72dae9bef` by treating the absent selector map like the existing no-host branch, invalidating any per-selector client and preserving global configuration; added `testHandlerSelectorWithEmptyHandle`.\n\nValidation passed: `./mvnw -B -ntp -pl shenyu-plugin/shenyu-plugin-logging/shenyu-plugin-logging-rabbitmq -am -Dtest=LoggingRabbitmqPluginDataHandlerTest -Dsurefire.failIfNoSpecifiedTests=false -Djacoco.skip=true -Dmaven.javadoc.skip=true -Drat.skip=true test` (8 tests; Checkstyle clean). Fresh hosted CI is now running, including the RabbitMQ E2E across the configured sync modes. -- 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]
