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: notifications-unsubscr...@logging.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to