milleruntime commented on a change in pull request #140: Updated MapReduce docs with 2.0 changes URL: https://github.com/apache/accumulo-website/pull/140#discussion_r245166347
########## File path: _docs-2/security/on-disk-encryption.md ########## @@ -78,8 +78,8 @@ its the additional data that gets encrypted on disk that could be exposed in a l ### Bulk Import -There are 2 ways to create RFiles for bulk ingest: with the [RFile API][rfile] and during Map Reduce using [AccumuloOutputFormat]. -The [RFile API][rfile] allows passing in the configuration properties for encryption mentioned above. The [AccumuloOutputFormat] does +There are 2 ways to create RFiles for bulk ingest: with the [RFile API][rfile] and during Map Reduce using [AccumuloFileOutputFormat]. +The [RFile API][rfile] allows passing in the configuration properties for encryption mentioned above. The [AccumuloFileOutputFormat] does not allow for encryption of RFiles so any data bulk imported through this process will be unencrypted. Review comment: Nice catch! ---------------------------------------------------------------- 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
