Aias00 opened a new pull request, #4144:
URL: https://github.com/apache/rocketmq-dashboard/pull/4144

   ## Summary
   
   - expose optional `traceTopic` in the canonical `rmq.message.trace` input 
schema
   - forward the supplied value through `MessageTraceToolHandler` to the 
existing four-argument `MessageService` path
   - preserve the current default behavior when the field is omitted or blank
   - add handler and catalog contract coverage
   
   ## Scope
   
   This change only extends message-id trace lookup. It does not add 
trace-by-key behavior, change the output schema, or introduce provider-specific 
logic. `MessageService` remains responsible for normalization and fallback to 
the legacy provider path.
   
   ## TDD evidence
   
   Before the implementation, the new tests failed because the catalog did not 
contain `traceTopic` and the handler invoked the three-argument service method. 
After the minimal implementation, both contracts pass.
   
   ## Verification
   
   - `mvn -B -ntp -f server/pom.xml 
-Dtest=MessageTraceToolHandlerTest,ToolCatalogTest,MessageServiceTest,ToolGatewayServiceTest
 test` — 58 tests passed
   - `mvn -B -ntp -f server/pom.xml -DskipTests package`
   - Checkstyle — 0 violations
   - `git diff --check`
   
   Closes #4143


-- 
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]

Reply via email to