sunnysabor opened a new pull request, #7150: URL: https://github.com/apache/shenyu/pull/7150
Fixes #6560 ### What changed - Replace the synthetic initialization record with Kafka producer topic metadata lookup. - Treat metadata lookup failure as initialization failure and close the producer. - Add regression tests for successful metadata lookup without send, and metadata timeout cleanup. ### Compatibility and impact The plugin no longer writes a synthetic record to the configured log topic. Initialization now waits for topic metadata and reports authentication, authorization, or metadata timeout errors synchronously. Normal request-log publishing is unchanged. ### Tests - PASS: `./mvnw -B -ntp -s /tmp/shenyu-maven-central-settings.xml -pl shenyu-plugin/shenyu-plugin-logging/shenyu-plugin-logging-kafka -am -Dtest=KafkaLogCollectClientTest -Dsurefire.failIfNoSpecifiedTests=false -DskipRemoteResources=true test` (2 tests, 0 failures; Checkstyle passed). - Full `./mvnw clean install -Dmaven.javadoc.skip=true` was attempted and stopped after 16 of 233 modules due to duration; no failure was observed in completed modules. ### Checklist - [x] Read the contribution guidelines. - [x] Added regression tests for the changed behavior. - [ ] Full `./mvnw clean install -Dmaven.javadoc.skip=true` passed locally. The full build was stopped after 16 of 233 modules due to duration; no failure was observed in completed modules. -- 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]
