jesmith17 commented on code in PR #3467:
URL: https://github.com/apache/logging-log4j2/pull/3467#discussion_r1959779986
##########
log4j-mongodb4/src/main/java/org/apache/logging/log4j/mongodb4/MongoDb4Connection.java:
##########
@@ -67,21 +67,23 @@ public MongoDb4Connection(
final ConnectionString connectionString,
final MongoClient mongoClient,
final MongoDatabase mongoDatabase,
+ final String collectionName,
Review Comment:
So the first problem is that the maven-wrapper breaks when running it on my
machine using maven 3.9.9
Specifically it complains about needing to update the
`distributionSha256Sum` which is impossible because 3.9.9. comes with a sha-512
sum. In order to get any of the `./mvnw` commands to run I had to rebuild the
wrapper, but I suspect you don't want me to check that in.
Seems like there is an undocumented dependent on a specific version of maven
that needs to be addressed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]