keith-turner commented on code in PR #5988:
URL: https://github.com/apache/accumulo/pull/5988#discussion_r2560771879
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -1280,7 +1280,9 @@ public enum Property {
"The durability used to write to the write-ahead log. Legal values are:"
+ " none, which skips the write-ahead log; log, which sends the data
to the"
+ " write-ahead log, but does nothing to make it durable; flush,
which pushes"
- + " data to the file system; and sync, which ensures the data is
written to disk.",
+ + " data to the file system; and sync, which ensures the data is
written to disk."
Review Comment:
> We would also need to fail the configuration if table durability is set to
sync and erasure coding is configured.
That behavior sounds really nice for the metadata table, would not want the
metadata table accidentally using EC because of the durability concerns. If
this not something that will be addressed in this PR then it would be good to
open an issue for this so its not lost, seems really important.
--
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]
