gerlowskija commented on code in PR #2239: URL: https://github.com/apache/solr/pull/2239#discussion_r1479699539
########## solr/solr-ref-guide/modules/deployment-guide/pages/backup-restore.adoc: ########## @@ -401,6 +401,17 @@ Any children under the `<repository>` tag are passed as additional configuration Information on each of the repository implementations provided with Solr is provided below. +By default, all the repository implementations verify the integrity of the index files before they are copied to the destination. However, it is possible to disable this integrity check by setting the optional configuration property `verifyChecksum`. + +`verifyChecksum`:: ++ +[%autowidth,frame=none] +|=== +|Optional |Default: `true` +|=== ++ +Defines whether the backup repository should check the index files integrity before copying them to the destination. Set `false` to disable the checksum verification, for example to verify the integrity in a different way. Review Comment: [0] Is it worth calling out the encryption scenario explicitly here? I can imagine users reading this and wondering why they would ever want to disable such a safety check... (If we're intentionally NOT mentioning encryption because it's a 'solr-sandbox' thing, I guess I understand that as well. Just figured I'd check) ########## solr/solr-ref-guide/modules/deployment-guide/pages/backup-restore.adoc: ########## @@ -422,7 +433,7 @@ LocalFileSystemRepository accepts the following configuration option: |=== + A valid file path (accessible to Solr locally) to use for backup storage and retrieval. -Used as a fallback when user's don't provide a `location` parameter in their Backup or Restore API commands +Used as a fallback when user's don't provide a `location` parameter in their Backup or Restore API commands. Review Comment: [0] There shouldn't be any apostrophe in "user's". -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
