oscerd opened a new pull request, #25032: URL: https://github.com/apache/camel/pull/25032
Backport of #25029 to `camel-4.18.x`. `Athena2QueryHelper.interrupted` gates both `shouldAttempt()` and `shouldWait()` but has no writer, so both guards are dead code. CAMEL-20297 had added the interrupt handling in `doWait()`; the CAMEL-22949 Task-API migration (`1b0fca17c0`) dropped the `catch (InterruptedException)` block that was its only writer. Because `ForegroundTask.run()` restores the interrupt status and `doWait()` discarded its return value, an interrupted poll loop spins with **no delay**, hammering `GetQueryExecution` until `waitTimeout` elapses. The fix records the interruption so both loops bail out. This branch carries the CAMEL-22949 migration, so it is affected. `camel-4.14.x` is **not** — it still has the original `Thread.sleep()` with its `catch` block, so no backport is needed there. Cherry-picked cleanly; `Athena2QueryHelperTest` passes 23/23 on this branch. No upgrade-guide entry — this restores intended behaviour with no user-visible configuration change. --- _Claude Code on behalf of oscerd_ -- 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]
