apurtell opened a new pull request, #2557:
URL: https://github.com/apache/phoenix/pull/2557

   `CDCQueryIT` can fail with `IllegalArgumentException: bound must be positive 
at CDCQueryIT.java:786`, where the test calls `rand.nextInt(lastDeletionTSpos - 
1)` to pick a random index into the unique timestamp list. `lastDeletionTSpos` 
is only assigned when the synthesized CDC change row stream contains at least 
one deletion encountered at a unique timestamp. Under the parameterized test 
matrix it can stay null, or be 0 or 1. The fix is to skip the random sub-range 
exercise when the available window is < 2.
   
   Co-authored-by: Claude Opus 4.8[1m] <[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]

Reply via email to