X-LightYear opened a new pull request, #4546:
URL: https://github.com/apache/rocketmq-dashboard/pull/4546

   ## Summary
   
   - prevent stale consumer progress requests from overwriting newer diagnostic 
results
   - track request ownership per instance/group diagnostic context
   - add deterministic regression coverage for overlapping progress requests
   
   ## Root cause
   
   `ConsumerPage.loadProgress` allowed overlapping requests for the same 
diagnostic context to update `progressByGroup` in completion order.
   
   An older request could therefore complete after a newer "Re-diagnose" 
request and overwrite the newer result.
   
   ## Fix
   
   Track the latest progress request generation per existing `cacheKey`.
   
   Only the request that still owns the latest generation is allowed to update 
progress or surface an error.
   
   ## Testing
   
   - added deterministic deferred-Promise regression coverage for stale 
progress responses
   - `ConsumerPageDiagnosticsRace.test.tsx`: 1 passed
   - `ConsumerPage.test.tsx`: 32 passed
   - frontend lint: 0 errors
   - `npm run build`: passed
   
   Fixes #4545


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