[jira] [Assigned] (IGNITE-321) CacheWriter methods should throw CacheWriterException

["Sergey Evdokimov (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/IGNITE-321?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sergey Evdokimov reassigned IGNITE-321:
---------------------------------------

    Assignee: Sergey Evdokimov  (was: Yakov Zhdanov)

> CacheWriter methods should throw CacheWriterException
> -----------------------------------------------------
>
>                 Key: IGNITE-321
>                 URL: https://issues.apache.org/jira/browse/IGNITE-321
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>            Reporter: Dmitriy Setrakyan
>            Assignee: Sergey Evdokimov
>            Priority: Blocker
>              Labels: wait_for_review
>             Fix For: sprint-2
>
>         Attachments: ignite-321.patch
>
>
> Tests where CacheWriter's methods write(All)/delete(All) throw an exception expect that Cache's methods put(All)/remove(All) throw CacheWriterException in this case.
> List of tests which expect CacheWriterException if CacheWriter throws an exception:
> {code}
> CacheWriterTest.shouldNotPutWhenWriteThroughFails
> CacheWriterTest.shouldWriteThoughUsingPutAll_partialSuccess
> CacheWriterTest.shouldNotRemoveWhenWriteThroughFails
> CacheWriterTest.shouldWriteThroughRemoveAll_partialSuccess
> CacheWriterTest.shouldWriteThroughRemoveAllSpecific_partialSuccess
> CacheWriterTest.shouldWrapWriterExceptions
> {code}
> We should make sure that these tests pass by catching IgniteException, calling hasCause(CacheWriterException.class), and if true, then throwing the CacheWriterException.



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