HarshMehta112 opened a new pull request, #9102:
URL: https://github.com/apache/gravitino/pull/9102
# Fixed #9085
## Summary
This PR updates the initialization logic in `LocalJobExecutor` to ensure
that `jobStatusKeepTimeInMs` is never set below a safe minimum threshold.
Previously, users could configure `JOB_STATUS_KEEP_TIME_MS` to values below
10 ms, which was unsupported and could lead to IllegalArgumentException.
With this change, any value below **10 ms** will now be **clamped to 10 ms**.
Values **≥ 10 ms** remain unchanged.
## Testing
- **Unit tests:** Added test verifying that values below 10 ms are clamped
correctly.
- **Existing tests:** Unaffected and continue to pass (values ≥ 10 ms behave
as before).
- **Manual verification:**
- Provided value `1 ms` → clamped to `10 ms`
- Provided value `11 ms` → remains `11 ms`
---
**Example Behavior:**
| Input (ms) | Result (ms) |
|-------------|-------------|
| 1 | 10 |
| 11 | 11 |
---
✅ **Result:** Safer initialization logic and consistent minimum timing
behavior in `LocalJobExecutor`.
--
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]
