[jira] [Commented] (IGNITE-3543) IGFS: Merge isRetryForSecondary() and verifyIntegrity() methods.

["Ivan Veselovsky (JIRA)" <ji...@apache.org>]

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

Ivan Veselovsky commented on IGNITE-3543:
-----------------------------------------

https://github.com/apache/ignite/pull/1420

> IGFS: Merge isRetryForSecondary() and verifyIntegrity() methods.
> ----------------------------------------------------------------
>
>                 Key: IGNITE-3543
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3543
>             Project: Ignite
>          Issue Type: Task
>          Components: IGFS
>    Affects Versions: 1.6
>            Reporter: Vladimir Ozerov
>            Assignee: Ivan Veselovsky
>             Fix For: 2.0
>
>
> There are two methods with very similar semantics:
> 1) {{IgfsPathIds.verifyIntegrity}}
> 2) {{IgfsMetaManager.isRetryForSecondary}}
> The latter method ensures that if path is incomplete, then the last existing item do not have reference to child with expected name, but unexpected ID. Semantically this situation means that concurrent update occurred. 
> Instead of heaving two identical methods, we should merge these checks in a single method {{IgfsPathIds.verifyIntegrity}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)