Review the new Camel Streamer (IGNITE-1790)

[Raul Kripalani]

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

[Denis Magda]

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
>

[Raul Kripalani-2]

Sure, that's fine.

Thanks, Denis.

Regards,
Raúl.
On 29 Oct 2015 08:50, "Denis Magda" <[hidden email]> 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
>>
>>
>

[Denis Magda]

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" <[hidden email]> 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
>>>
>>>

[Denis Magda]

Hi Raul,

Is there any chance we make this contribution available as a part of the upcoming release?

—
Denis

> On 4 нояб. 2015 г., at 17:29, Denis Magda <[hidden email]> wrote:
>
> 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" <[hidden email]> 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
>>>>
>>>>
>

[Raul Kripalani]

On Mon, Nov 16, 2015 at 12:32 PM, Denis Magda <[hidden email]> wrote:

> Is there any chance we make this contribution available as a part of the
> upcoming release?
>

Yes, it'll be. Will get around to reviewing it by tomorrow.

*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