Open rnithinpaladin opened 1 month ago
In the JobScheduler.java
file, the noOfBatches
field was removed, and the schedulePluginShipperJobs
method was modified to reflect changes in the scheduling logic. This update simplifies the class by removing unnecessary fields and potentially improves the efficiency of job scheduling.
File Path | Change Summary |
---|---|
jobs/job-scheduler/src/main/java/.../JobScheduler.java |
Removed noOfBatches field and updated schedulePluginShipperJobs method for scheduling logic. |
In the land of code where schedules reign,
A field callednoOfBatches
faced its bane.
With logic refined and cleaner slate,
Jobs now ship at a faster rate.
🐇✨ Hopping through the code with glee,
The scheduler's now as light as can be!
[!WARNING]
Review ran into problems
Problems (1)
* Git: Failed to clone repository. Please contact CodeRabbit support.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Pr merged in release
Description
Removed number of batch and vulnerability shipper
Problem
Solution
Fixes # (issue if any)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:
Other Information:
List any documentation updates that are needed for the Wiki
Summary by CodeRabbit
noOfBatches
field, streamlining theschedulePluginShipperJobs
method for better performance and maintainability.