stefan-egli commented on a change in pull request #283:
URL: https://github.com/apache/jackrabbit-oak/pull/283#discussion_r609570848



##########
File path: 
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongoSessionFactory.java
##########
@@ -37,22 +37,23 @@
 
     private final ClientSessionOptions options;
 
+    private final MongoClock clock;

Review comment:
       I might not fully understand this, but MongoClock is somewhat confusing 
me. IIUC it is only used by MongoSessionFactory and serves as a replacement for 
the `clusterTime` and `operationTime` fields that were previously used (and now 
become obsolete). In that case, could the scope of `MongoClock` not actually be 
limited to be a private/inner class of MongoSessionFactory - or is the 
MongoClock indeed being used outside of this class?




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to