ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ivan Veselovsky (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-3543) IGFS: Merge isRetryForSecondary() and verifyIntegrity() methods.
Date Wed, 11 Jan 2017 11:16:59 GMT

    [ 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)

Mime
View raw message