You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Raul Kripalani <ra...@apache.org> on 2015/10/28 13:51:29 UTC
Review the new Camel Streamer (IGNITE-1790)
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)
Posted by Raul Kripalani <ra...@apache.org>.
On Mon, Nov 16, 2015 at 12:32 PM, Denis Magda <dm...@gridgain.com> 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)
Posted by Denis Magda <dm...@gridgain.com>.
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 <dm...@gridgain.com> 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" <dm...@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
>>>>
>>>>
>
Re: Review the new Camel Streamer (IGNITE-1790)
Posted by Denis Magda <dm...@gridgain.com>.
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" <dm...@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
>>>
>>>
Re: Review the new Camel Streamer (IGNITE-1790)
Posted by Raul Kripalani <ra...@evosent.com>.
Sure, that's fine.
Thanks, Denis.
Regards,
Raúl.
On 29 Oct 2015 08:50, "Denis Magda" <dm...@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
>>
>>
>
Re: Review the new Camel Streamer (IGNITE-1790)
Posted by Denis Magda <dm...@gridgain.com>.
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
>