[
https://issues.apache.org/jira/browse/IGNITE-1971?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roman Shtykh updated IGNITE-1971:
---------------------------------
Component/s: streaming
> Refactor various streamers implementations
> ------------------------------------------
>
> Key: IGNITE-1971
> URL: https://issues.apache.org/jira/browse/IGNITE-1971
> Project: Ignite
> Issue Type: Bug
> Components: streaming
> Affects Versions: 1.5
> Reporter: Denis Magda
> Fix For: 1.6
>
>
> Re-visit streamers implementations and should address at least the following
> points below.
> 1) {{start}} method in streamers implementations throws
> {{IllegalArgumentException}}.
> According to the contract of IllegalArgumentException this exception is
> thrown only when an invalid argument is passed into the function. However no
> argument is passed to the method at all and we shouldn't throw this exception
> from this point of execution.
> My suggestion is to use org.apache.ignite.IgniteIllegalStateException if we
> don't find better alternative. See how it's used in Ignite.ignite() method
> description.
> 2) Streamers perform the same basics checks in their implementation of
> {{start}} method.
> It makes sense to add {{start}} method to StreamAdapter and perform there
> basic checks that are the same for every streamer.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
