[ https://issues.apache.org/jira/browse/OAK-7223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16346726#comment-16346726 ]
Marcel Reutegger commented on OAK-7223: --------------------------------------- I would implement the new utility method slightly different. This avoids the issue with catching all types of exceptions but then wrapping it with an IOException (even if the original one was, as in most cases, an IOException). See attached [^OAK-7223-3.patch]. It would also be good to have a test case that verifies the expected behaviour. > Files could be kept partially in case of disconnection from backends > -------------------------------------------------------------------- > > Key: OAK-7223 > URL: https://issues.apache.org/jira/browse/OAK-7223 > Project: Jackrabbit Oak > Issue Type: Bug > Components: blob-plugins > Affects Versions: 1.8.1 > Reporter: Amit Jain > Assignee: Amit Jain > Priority: Blocker > Fix For: 1.10, 1.8.2 > > Attachments: OAK-7223-2.patch, OAK-7223-3.patch, OAK-7223.patch > > > The FileCache#load method needs to clean the files which may have been > downloaded partially in case of errors from backend. This partially > downloaded file should be removed before returning from the method. -- This message was sent by Atlassian JIRA (v7.6.3#76005)