mreutegg commented on code in PR #858:
URL: https://github.com/apache/jackrabbit-oak/pull/858#discussion_r1129336932
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/Document.java:
##########
@@ -151,6 +151,22 @@ public void seal() {
}
}
+ /**
+ * UnSeals this document and turns it into a mutable object.
+ *
+ * @see #seal()
+ */
+ void unSeal() {
Review Comment:
I don't like adding method. The idea behind sealing a document is to ensure
it becomes immutable. A DocumentStore implementation can use this to construct
a Document, put it into a cache and then return it to a caller. With this new
method a caller can modify a sealed Document, violating a basic principle.
--
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]
