Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
Subject: Re: Review the new Camel Streamer (IGNITE-1790)
To: dev@ignite.apache.org
References: 
 <CAKoNANvXtTeGgkV0RJzQFhMaTbFDcmqEA+9n0T3_fSoUmFNOUw@mail.gmail.com>
 <5631DDCA.3000801@gridgain.com>
 <CAKoNANvspuSEzZO3TfbzXXt1wekTJ8ZH6uE=RDtQf2CpbH6UiA@mail.gmail.com>
From: Denis Magda <dmagda@gridgain.com>
Message-ID: <563A1655.2030202@gridgain.com>
Date: Wed, 4 Nov 2015 17:29:41 +0300
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101
 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: 
 <CAKoNANvspuSEzZO3TfbzXXt1wekTJ8ZH6uE=RDtQf2CpbH6UiA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit

Raul, sorry for the delay.

I've reviewed your implementation and left minor comments in the ticket.

--
Thanks,
Denis

On 10/29/2015 12:17 PM, Raul Kripalani wrote:
> Sure, that's fine.
>
> Thanks, Denis.
>
> Regards,
> Raúl.
> On 29 Oct 2015 08:50, "Denis Magda" <dmagda@gridgain.com> wrote:
>
>> Hi Raul,
>>
>> I'll be able to take a look at the changes. However this should happen not
>> earlier than in the beginning of the next week.
>>
>> Does it work for you?
>>
>> --
>> Denis
>>
>> On 10/28/2015 3:51 PM, Raul Kripalani wrote:
>>
>>> Hello,
>>>
>>> I've pushed a new Apache Camel streamer in the ignite-1790 branch.
>>>
>>> Build and tests ran successfully on TC (Ignite Streamers build config).
>>>
>>> Could someone please review it before I merge it into master?
>>>
>>> Thanks,
>>>
>>> *Raúl Kripalani*
>>> PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and
>>> Messaging Engineer
>>> http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani
>>> http://blog.raulkr.net | twitter: @raulvk
>>>
>>>