pcd1193182 commented on this pull request.


> @@ -1054,6 +1054,8 @@ static const char *spa_feature_names[] = {
        "com.delphix:embedded_data",
        "org.open-zfs:large_blocks",
        "org.illumos:sha512",
+       "com.delphix:redaction_bookmarks",

It's primarily for tracking purposes. By having a featureflag for each one, we 
can easily track whether each feature is active or enabled, which allows pools 
that have had redaction bookmarks (or redacted datasets) added and then deleted 
to be read by older systems.  The two features are also fundamentally 
differently implemented, so it makes sense for them to be tracked separately 
even if they're introduced by the same code to support the same workflow.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/484#discussion_r152368014
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/T1fcfd4fc81bcdfff-M1aecea42b8cb7c30e8099456
Powered by Topicbox: https://topicbox.com

Reply via email to