Mailing-List: contact issues-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
Date: Tue, 11 Apr 2017 08:07:42 +0000 (UTC)
From: "Vladimir Ozerov (JIRA)" <jira@apache.org>
To: issues@ignite.apache.org
Message-ID: <JIRA.12914833.1448023181000.254196.1491898062673@Atlassian.JIRA>
In-Reply-To: <JIRA.12914833.1448023181000@Atlassian.JIRA>
References: <JIRA.12914833.1448023181000@Atlassian.JIRA> <JIRA.12914833.1448023181508@jira-lw-us.apache.org>
Subject: [jira] [Updated] (IGNITE-1971) Refactor various streamers
 implementations
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Tue, 11 Apr 2017 08:07:48 -0000


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

Vladimir Ozerov updated IGNITE-1971:
------------------------------------
    Fix Version/s:     (was: 2.0)
                   2.1

> 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.0.final
>            Reporter: Denis Magda
>             Fix For: 2.1
>
>
> 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.15#6346)