basic mongodb indexig added#1264
Conversation
📝 WalkthroughWalkthroughThis PR adds MongoDB index initialization to the startup lifecycle. MongoDbAwareService gains contract methods for index creation, multiple services declare their indexes, a handler orchestrates initialization across all services and databases, and the startup bean wires everything together during application startup. ChangesMongoDB Index Initialization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: b8f79e69-de84-4eef-9111-d9225b2bf4c3
📒 Files selected for processing (6)
software/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/scheduled/LifecycleBean.javasoftware/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/scheduled/MongoInitializationHandler.javasoftware/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/InventoryItemService.javasoftware/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/ItemListService.javasoftware/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/MongoDbAwareService.javasoftware/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/StorageBlockService.java
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: CI-Pipeline / Build
🔇 Additional comments (4)
software/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/InventoryItemService.java (1)
16-16: LGTM!Also applies to: 429-434
software/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/service/mongo/ItemListService.java (1)
7-7: LGTM!Also applies to: 16-16, 69-74
software/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/scheduled/MongoInitializationHandler.java (1)
11-27: LGTM!software/core/oqm-core-api/src/main/java/tech/ebp/oqm/core/api/scheduled/LifecycleBean.java (1)
46-47: LGTM!Also applies to: 151-152
GregJohnStewart
left a comment
There was a problem hiding this comment.
Overall, nice start. Just some polish for the final form
0462e2d to
d184c27
Compare
GregJohnStewart
left a comment
There was a problem hiding this comment.
One last bit... in the theme of learning about best practices
f8816b1
into
Epic-Breakfast-Productions:development

Checklist:
issue #301 (and #1259)
Summary by CodeRabbit
New Features
Chores