Re: Review the new Camel Streamer (IGNITE-1790)

2015-11-17 Thread Raul Kripalani 
On Mon, Nov 16, 2015 at 12:32 PM, Denis Magda  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

Re: Review the new Camel Streamer (IGNITE-1790)

2015-11-16 Thread 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  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"  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
 
 
>

Re: Review the new Camel Streamer (IGNITE-1790)

2015-11-04 Thread 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"  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

Re: Review the new Camel Streamer (IGNITE-1790)

2015-10-29 Thread Raul Kripalani 
Sure, that's fine.

Thanks, Denis.

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

Re: Review the new Camel Streamer (IGNITE-1790)

2015-10-29 Thread 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