-
Notifications
You must be signed in to change notification settings - Fork 3
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
Mongo7 upgrade #394
Mongo7 upgrade #394
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #394 +/- ##
=============================================
+ Coverage 94.08% 94.10% +0.02%
- Complexity 1594 1595 +1
=============================================
Files 102 102
Lines 4734 4735 +1
Branches 827 828 +1
=============================================
+ Hits 4454 4456 +2
+ Misses 266 265 -1
Partials 14 14 |
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.
Looks pretty good, just needs retryWrites and I have 1 question
@@ -107,9 +107,10 @@ public void startUpWith2ConfigDocs() throws Exception { | |||
|
|||
final Pattern errorPattern = Pattern.compile( | |||
"Failed to create index: Write failed with error code 11000 and error message " + | |||
"'(exception: )?E11000 duplicate key error (index|collection): " + | |||
"'(exception: )?(.*)E11000 duplicate key error (index|collection): " + |
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.
What is captured by (.*)
?
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.
final MongoClientSettings.Builder mongoBuilder = MongoClientSettings.builder().applyToClusterSettings( | ||
builder -> builder.hosts(Arrays.asList(new ServerAddress(c.getMongoHost())))); |
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.
You're going to need the retryWrites parameter as well
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.
Adding the retryWrites parameter will be included in the next PR.
No description provided.