stefan-egli commented on code in PR #1294:
URL: https://github.com/apache/jackrabbit-oak/pull/1294#discussion_r1478443657
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreService.java:
##########
@@ -185,7 +186,12 @@ public class DocumentNodeStoreService {
/**
* Feature toggle name to enable detailed GC for Mongo Document Store
*/
- private static final String FT_NAME_DEATILED_GC =
"FT_DETAILED_GC_OAK-10199";
+ private static final String FT_NAME_DETAILED_GC =
"FT_DETAILED_GC_OAK-10199";
+
+ /**
+ * Feature toggle name to enable embedded verification for detailed GC
mode for Mongo Document Store
+ */
+ private static final String FT_NAME_EMBEDDED_VERIFICATION =
"FT_EMBEDDED_VERIFICATION_OAK-10633";
Review Comment:
Starting to think that we might have to revisit our use of feature toggle
for DocumentNodeStore (besides the fact that we still have legacy system
properties) : it looks like we more and more opt to not only introduce new
config options, but we also make them increasingly available as feature toggle.
Which makes our list of feature toggles at some point fairly large.
Not saying that we shouldn't introduce this one - just noting that we make
excessive use of it lately.
##########
oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentNodeStoreBuilder.java:
##########
@@ -138,4 +138,26 @@ public RDBDocumentNodeStoreBuilder
setDocStoreDetailedGCFeature(@Nullable Featur
public Feature getDocStoreDetailedGCFeature() {
return null;
}
+
+ @Override
+ public boolean isEmbeddedVerificationEnabled() {
+ return false;
Review Comment:
what was again the reason why detailedGC is not supported for RDB?
--
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: dev-unsubscr...@jackrabbit.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
