[jira] [Assigned] (IGNITE-3543) IGFS: Merge isRetryForSecondary() and verifyIntegrity() methods.
[ https://issues.apache.org/jira/browse/IGNITE-3543?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vladimir Ozerov reassigned IGNITE-3543: --- Assignee: Ivan Veselovsky (was: Vladimir Ozerov) Ivan, If there methods are not identical semantically, then I propose to simply close the ticket as "Won't fix". > 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: 1.9 > > > 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)
[jira] [Assigned] (IGNITE-3543) IGFS: Merge isRetryForSecondary() and verifyIntegrity() methods.
[ https://issues.apache.org/jira/browse/IGNITE-3543?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ivan Veselovsky reassigned IGNITE-3543: --- Assignee: Ivan Veselovsky > 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)