Aias00 commented on issue #6523: URL: https://github.com/apache/shenyu/issues/6523#issuecomment-5157898040
Follow-up verification note (2026-08-02 audit) — severity clarification. The described behavior is accurate: `HttpSyncDataService.java:280-296` has no `break` after a successful `doLongPolling(server)`, so the `for (time=1; time<=retryTimes; time++)` loop calls `doLongPolling` up to 10 times per outer cycle, and the retry counter only short-circuits on exception via `continue`. One severity caveat: `doLongPolling` blocks server-side until a config change or the long-poll timeout, so the missing `break` does **not** produce extra outbound traffic or busy-waiting. The real issue is that the `retryTimes` counter becomes semantically meaningless on the success path and the inter-poll pacing is removed. Still worth fixing (add `break` after success), but lower impact than "repeats polls ten times per loop" might imply. -- 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]
