ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Ozerov (JIRA)" <j...@apache.org>
Subject [jira] [Created] (IGNITE-3668) IGFS: Local secondary: Review all "throw new IOException" places.
Date Wed, 10 Aug 2016 07:28:20 GMT
Vladimir Ozerov created IGNITE-3668:
---------------------------------------

             Summary: IGFS: Local secondary: Review all "throw new IOException" places.
                 Key: IGNITE-3668
                 URL: https://issues.apache.org/jira/browse/IGNITE-3668
             Project: Ignite
          Issue Type: Sub-task
          Components: IGFS
    Affects Versions: 1.7
            Reporter: Vladimir Ozerov
            Assignee: Taras Ledkov
             Fix For: 1.8


We have several places in {{LocalIgfsSecondaryFileSystem}} where {{IOException}} is thrown.
This is not correct, because:
1) Exception must be specific. E.g. "path-not-found-exception".
2) It doesn't make sense to throw {{IOException}} and cast it to {{IgfsException}} immediately.

Also we should not throw {{IgniteException}}. Proper {{IgfsException}} must be thrown.

Let's review all these places and replace {{IOException}} with proper {{IgfsException}} inheritor.


Successful implementation should not introduce any new test failures.



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

Mime
View raw message