davsclaus opened a new pull request, #24755: URL: https://github.com/apache/camel/pull/24755
## Summary - **Fix `readLockTimeout=0` regression** in 3 strategies (`FtpChangedExclusiveReadLockStrategy`, `FilesChangedExclusiveReadLockStrategy`, `SmbChangedExclusiveReadLockStrategy`) — the CAMEL-17121 Tasks API conversion (Camel 3.14) broke the documented "0 means forever" contract because `TimeBoundedBudget` only recognizes `-1` as unlimited, not `0`. Now uses `withUnlimitedDuration()` when `timeout <= 0`. - **Convert `SftpChangedExclusiveReadLockStrategy`** from old while-loop + `StopWatch` + `Thread.sleep` to the Tasks API for consistency with the FTP variant, including the timeout fix. - **Extract `SftpExclusiveReadLockCheck`** class (mirrors the existing `ExclusiveReadLockCheck` used by FTP). - **Fix copy-paste task name** in Azure Files strategy (`"ftp-..."` → `"azure-files-..."`) ## Test plan - [x] New unit test `FtpChangedReadLockTimeoutZeroTest` (3 tests: timeout=0, timeout=-1, timeout=positive) — all pass - [x] `camel-ftp` builds cleanly - [x] `camel-azure-files` builds cleanly - [x] `camel-smb` builds cleanly _Claude Code on behalf of davsclaus_ Co-Authored-By: Claude Opus 4.6 <[email protected]> -- 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]
