weillercarvalho commented on issue #6609: URL: https://github.com/apache/jmeter/issues/6609#issuecomment-3552979052
You're right @vlsi , makes more sense, quick explanation of the new changes: I ditched the inclusive “max” change and went back to Java‑style semantics: start date is inclusive, end date is exclusive. To keep single‑day ranges working I still handle `start == end` as a special case so you get that exact day without blowing up. The helper is just `ThreadLocalRandom.nextLong(start,end)` again, no `max+1`. Tests were rolled back to expect the start date in those locale examples and they now assert `result < max`. Docs got tweaked to call out that end is exclusive and the changes log mentions the behavior shift. -- 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]
