dlmarion commented on code in PR #6219:
URL: https://github.com/apache/accumulo/pull/6219#discussion_r2940685728
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -391,6 +391,11 @@ public enum Property {
"Maximum number of threads the TabletGroupWatcher will use in its
BatchScanner to"
+ " look for tablets that need maintenance.",
"2.1.4"),
+ MANAGER_TABLE_DELETE_OPTIMIZATION("manager.table.delete.optimization",
"true",
+ PropertyType.BOOLEAN,
+ "Scans the metadata table for tablet references so creation of delete
markers can be skipped"
+ + "This behavior can now be disabled, allowing the GC to handle the
deletion of files.",
Review Comment:
May want to explain when the user might want to set this to false.
```suggestion
"When deleting a table the Manager will remove related table
directories from "+
" the storage volumes if there are no other references to the files in
the "+
" metadata table. When deleting a lot of tables this optimization can
be costly. " +
" Setting this value to false will skip this optimization and the
table directory " +
" cleanup will occur in the Garbage Collector instead.",
```
--
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]
