mikewalch commented on a change in pull request #560: Provide new Crypto 
interface & impl
URL: https://github.com/apache/accumulo/pull/560#discussion_r202405087
 
 

 ##########
 File path: core/src/main/java/org/apache/accumulo/core/conf/Property.java
 ##########
 @@ -867,6 +797,20 @@
           + " To add a summarizer set "
           + "`table.summarizer.<unique id>=<summarizer class name>.` If the 
summarizer has options"
           + ", then for each option set" + " `table.summarizer.<unique 
id>.opt.<key>=<value>`."),
+  // Crypto-related properties
+  @Experimental
+  TABLE_CRYPTO_PREFIX("table.crypto.opts.", null, PropertyType.PREFIX,
+      "Properties related to on-disk file encryption."),
+  @Experimental
+  @Sensitive
+  TABLE_CRYPTO_SENSITIVE_PREFIX("table.crypto.opts.sensitive.", null, 
PropertyType.PREFIX,
+      "Sensitive properties related to on-disk file encryption."),
+  @Experimental
+  TABLE_CRYPTO_SERVICE("table.crypto.service",
+      "org.apache.accumulo.core.security.crypto.impl.NoCryptoService", 
PropertyType.CLASSNAME,
+      "The class which executes on-disk file encryption. The default does 
nothing. To enable "
+          + "encryption, replace this classname with an implementation of the"
+          + "org.apache.accumulo.core.security.crypto.CryptoService 
interface."),
 
 Review comment:
   could use `{% jlink -f 
org.apache.accumulo.core.security.crypto.CryptoService %}`

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to