[ 
https://issues.apache.org/jira/browse/LUCENE-8843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16860673#comment-16860673
 ] 

ASF subversion and git services commented on LUCENE-8843:
---------------------------------------------------------

Commit 1ed253460fe9d9722d31602c2a4f827862baff89 in lucene-solr's branch 
refs/heads/branch_8x from Adrien Grand
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=1ed2534 ]

LUCENE-8843: Add CHANGES entry.


> Only ignore IOException on dirs when invoking force
> ---------------------------------------------------
>
>                 Key: LUCENE-8843
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8843
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: core/other
>            Reporter: Jason Tedor
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> I think the IOException suppression in IOUtils#fsync when fsyncing 
> directories is too broad, it suppresses any IOException that occurs in this 
> method if isDir is true. For example, it causes exceptions opening 
> non-existent directories, if access is denied to the directory, or general 
> filesystem errors to be suppressed. To me that feels like it could be 
> suppressing application bugs, or at least exceptions an application would 
> want to know about. Instead, the IOException suppression should only apply 
> after we have successfully opened the directory. I submitted a PR to propose 
> a change along these lines: [https://github.com/apache/lucene-solr/pull/706]. 
> Please let me know what you think.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to