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

   ## What is the purpose of the change
   
   Fix producer connection queries so Studio passes the required producer group 
and topic to the RocketMQ admin API, avoiding silent empty results on RocketMQ 
5.3.3.
   
   ## Brief changelog
   
   - Require both topic and producer group for producer connection queries.
   - Pass `producerGroup` and `topic` to `examineProducerConnectionInfo`.
   - Discover producer clients from Broker producer tables for the general 
client list.
   - Return HTTP 400 for missing parameters and HTTP 502 for RocketMQ query 
failures.
   - Add backend and frontend regression tests for the updated behavior.
   
   ## Verifying this change
   
   - `cd server && mvn 
-Dtest=ProducerConnectionServiceTest,ProducerControllerTest,RocketMQClientProviderTest,ClientProviderStubTest
 test`
   - `cd server && mvn test`
   - `cd server && mvn -DskipTests package`
   - `cd web && npm test -- src/api/producer.test.ts 
src/pages/studio/__tests__/Producer.test.tsx --maxWorkers=1 
--no-file-parallelism`
   - `cd web && npm test -- --maxWorkers=1 --no-file-parallelism`
   - `cd web && npm run build`
   - `git diff --check`


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