-
Notifications
You must be signed in to change notification settings - Fork 68
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
feature(api): add refiller index goroutine for search index #907
Conversation
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## master #907 +/- ##
==========================================
- Coverage 68.93% 68.38% -0.56%
==========================================
Files 195 197 +2
Lines 10866 11050 +184
==========================================
+ Hits 7491 7556 +65
- Misses 2934 3050 +116
- Partials 441 444 +3
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
/build |
/build |
/build |
/build |
/build |
/build |
/build |
Build and push Docker images with tag: feature-add-refiller-goroutine.2023-09-12.856b735 |
2 similar comments
Build and push Docker images with tag: feature-add-refiller-goroutine.2023-09-12.856b735 |
Build and push Docker images with tag: feature-add-refiller-goroutine.2023-09-12.856b735 |
@@ -84,6 +84,7 @@ func main() { | |||
databaseSettings := applicationConfig.Redis.GetSettings() | |||
notificationHistorySettings := applicationConfig.NotificationHistory.GetSettings() | |||
database := redis.NewDatabase(logger, databaseSettings, notificationHistorySettings, redis.API) | |||
database.Flush() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ой
Refiller goroutine
Repopulates the search index thereby removing memory leaks that appeared when filling, updating and searching the index. Also removed memory leaks in tests and refactoring
TODO: add tests for refiller