iprithv opened a new pull request, #4776:
URL: https://github.com/apache/solr/pull/4776

   When an incremental collection backup fails during shard copy, `BackupCmd` 
logs the real `SolrException` and then tries to clean up the partial backup. 
Cleanup deletes `zk_backup_<id>`, which is only created *after* shard copy.
   
   If shard copy fails first, that directory does not exist. 
`LocalFileSystemRepository.deleteDirectory()` goes through NIO/`PathUtils` and 
throws `java.nio.file.NoSuchFileException`. Cleanup only caught 
`java.io.FileNotFoundException`, which is a different type, so the not-found 
exception replaced the original backup error in the API response.
   
   Operators then see a bare missing-path error instead of the actual cause 
(allow-paths, no leader, shard copy failure, etc.).


-- 
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]

Reply via email to