-
Notifications
You must be signed in to change notification settings - Fork 438
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve Edge Caching Mechanism in InitializeEdgesWorker #2827
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #2827 +/- ##
=============================================
- Coverage 57.01% 56.98% -0.03%
+ Complexity 9659 9649 -10
=============================================
Files 2262 2262
Lines 96538 96556 +18
Branches 7126 7128 +2
=============================================
- Hits 55031 55011 -20
- Misses 39460 39498 +38
Partials 2047 2047 |
@michaelgrill: Can you please take over?
@Sn0w3y: I suggest we make the refresh time configurable (time in minutes). What do you think? |
I will add a Configurable Time to the Worker aswell as "Enable and Disable" Option :) |
One value is sufficient. Negative or zero means "disable". |
Done :) |
I had a quick look and renamed the config property to |
@Sn0w3y Thank you so much! We’ve started testing it internally! |
Does it work? Could you please tell us if the PR is successful ? |
This pull request enhances the InitializeEdgesWorker class by implementing a regular caching mechanism for edges from PostgreSQL. The main improvements are:
Regular edge caching: Edges are now cached immediately upon startup and then every 20 minutes.
Simplified scheduling: Uses a ScheduledExecutorService to manage both immediate and periodic tasks.
Executor management: Ensures that executors are properly shut down when stopping the worker.
Improved error handling and logging: Enhanced logging for better error tracing and debugging.
Benefits