Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Merged. Cheers, Anjana. On Fri, Feb 26, 2016 at 4:37 PM, Supun Sethunga wrote: > Added the changes as per code review. Please merge [1]. > > [1] https://github.com/wso2/carbon-analytics/pull/159 > > Thanks, > Supun > > On Fri, Feb 26, 2016 at 11:30 AM, Anjana Fernando wrote: > >> Please schedule a code review for this. >> >> Cheers, >> Anjana. >> >> On Fri, Feb 26, 2016 at 11:03 AM, Supun Sethunga wrote: >> >>> Hi, >>> >>> Please find the PR [1] for adding string compression to the event >>> analytics relational provider. >>> >>> [1] https://github.com/wso2/carbon-analytics/pull/159 >>> >>> Regards, >>> Supun >>> >>> On Wed, Feb 24, 2016 at 3:35 PM, Supun Sethunga wrote: >>> Thanks! Regards, Supun On Wed, Feb 24, 2016 at 2:20 PM, Anjana Fernando wrote: > Merged the PR. > > Cheers, > Anjana. > > On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga > wrote: > >> Hi, >> >> I have done the changes suggested during the code review. Please >> review and merge [1]. >> >> Will send a separate PR for Compressing the json string. >> >> >> Thanks, >> Supun >> >> On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando >> wrote: >> >>> Hi Supun, >>> >>> Can you please schedule a code review for this. >>> >>> Cheers, >>> Anjana. >>> >>> On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga >>> wrote: >>> Hi, Attached [1] is the PR for $subject. Please review and merge. [1] https://github.com/wso2/carbon-analytics/pull/156 Thanks, Supun -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 >>> >>> >>> >>> -- >>> *Anjana Fernando* >>> Senior Technical Lead >>> WSO2 Inc. | http://wso2.com >>> lean . enterprise . middleware >>> >> >> >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 >>> >>> >>> >>> -- >>> *Supun Sethunga* >>> Software Engineer >>> WSO2, Inc. >>> http://wso2.com/ >>> lean | enterprise | middleware >>> Mobile : +94 716546324 >>> >> >> >> >> -- >> *Anjana Fernando* >> Senior Technical Lead >> WSO2 Inc. | http://wso2.com >> lean . enterprise . middleware >> > > > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Added the changes as per code review. Please merge [1]. [1] https://github.com/wso2/carbon-analytics/pull/159 Thanks, Supun On Fri, Feb 26, 2016 at 11:30 AM, Anjana Fernando wrote: > Please schedule a code review for this. > > Cheers, > Anjana. > > On Fri, Feb 26, 2016 at 11:03 AM, Supun Sethunga wrote: > >> Hi, >> >> Please find the PR [1] for adding string compression to the event >> analytics relational provider. >> >> [1] https://github.com/wso2/carbon-analytics/pull/159 >> >> Regards, >> Supun >> >> On Wed, Feb 24, 2016 at 3:35 PM, Supun Sethunga wrote: >> >>> Thanks! >>> >>> Regards, >>> Supun >>> >>> On Wed, Feb 24, 2016 at 2:20 PM, Anjana Fernando >>> wrote: >>> Merged the PR. Cheers, Anjana. On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga wrote: > Hi, > > I have done the changes suggested during the code review. Please > review and merge [1]. > > Will send a separate PR for Compressing the json string. > > > Thanks, > Supun > > On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando > wrote: > >> Hi Supun, >> >> Can you please schedule a code review for this. >> >> Cheers, >> Anjana. >> >> On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga >> wrote: >> >>> Hi, >>> >>> Attached [1] is the PR for $subject. Please review and merge. >>> >>> [1] https://github.com/wso2/carbon-analytics/pull/156 >>> >>> Thanks, >>> Supun >>> >>> -- >>> *Supun Sethunga* >>> Software Engineer >>> WSO2, Inc. >>> http://wso2.com/ >>> lean | enterprise | middleware >>> Mobile : +94 716546324 >>> >> >> >> >> -- >> *Anjana Fernando* >> Senior Technical Lead >> WSO2 Inc. | http://wso2.com >> lean . enterprise . middleware >> > > > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware >>> >>> >>> >>> -- >>> *Supun Sethunga* >>> Software Engineer >>> WSO2, Inc. >>> http://wso2.com/ >>> lean | enterprise | middleware >>> Mobile : +94 716546324 >>> >> >> >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Please schedule a code review for this. Cheers, Anjana. On Fri, Feb 26, 2016 at 11:03 AM, Supun Sethunga wrote: > Hi, > > Please find the PR [1] for adding string compression to the event > analytics relational provider. > > [1] https://github.com/wso2/carbon-analytics/pull/159 > > Regards, > Supun > > On Wed, Feb 24, 2016 at 3:35 PM, Supun Sethunga wrote: > >> Thanks! >> >> Regards, >> Supun >> >> On Wed, Feb 24, 2016 at 2:20 PM, Anjana Fernando wrote: >> >>> Merged the PR. >>> >>> Cheers, >>> Anjana. >>> >>> On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga wrote: >>> Hi, I have done the changes suggested during the code review. Please review and merge [1]. Will send a separate PR for Compressing the json string. Thanks, Supun On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando wrote: > Hi Supun, > > Can you please schedule a code review for this. > > Cheers, > Anjana. > > On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga > wrote: > >> Hi, >> >> Attached [1] is the PR for $subject. Please review and merge. >> >> [1] https://github.com/wso2/carbon-analytics/pull/156 >> >> Thanks, >> Supun >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 >>> >>> >>> >>> -- >>> *Anjana Fernando* >>> Senior Technical Lead >>> WSO2 Inc. | http://wso2.com >>> lean . enterprise . middleware >>> >> >> >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Hi, Please find the PR [1] for adding string compression to the event analytics relational provider. [1] https://github.com/wso2/carbon-analytics/pull/159 Regards, Supun On Wed, Feb 24, 2016 at 3:35 PM, Supun Sethunga wrote: > Thanks! > > Regards, > Supun > > On Wed, Feb 24, 2016 at 2:20 PM, Anjana Fernando wrote: > >> Merged the PR. >> >> Cheers, >> Anjana. >> >> On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga wrote: >> >>> Hi, >>> >>> I have done the changes suggested during the code review. Please review >>> and merge [1]. >>> >>> Will send a separate PR for Compressing the json string. >>> >>> >>> Thanks, >>> Supun >>> >>> On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando >>> wrote: >>> Hi Supun, Can you please schedule a code review for this. Cheers, Anjana. On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga wrote: > Hi, > > Attached [1] is the PR for $subject. Please review and merge. > > [1] https://github.com/wso2/carbon-analytics/pull/156 > > Thanks, > Supun > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware >>> >>> >>> >>> -- >>> *Supun Sethunga* >>> Software Engineer >>> WSO2, Inc. >>> http://wso2.com/ >>> lean | enterprise | middleware >>> Mobile : +94 716546324 >>> >> >> >> >> -- >> *Anjana Fernando* >> Senior Technical Lead >> WSO2 Inc. | http://wso2.com >> lean . enterprise . middleware >> > > > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Thanks! Regards, Supun On Wed, Feb 24, 2016 at 2:20 PM, Anjana Fernando wrote: > Merged the PR. > > Cheers, > Anjana. > > On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga wrote: > >> Hi, >> >> I have done the changes suggested during the code review. Please review >> and merge [1]. >> >> Will send a separate PR for Compressing the json string. >> >> >> Thanks, >> Supun >> >> On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando wrote: >> >>> Hi Supun, >>> >>> Can you please schedule a code review for this. >>> >>> Cheers, >>> Anjana. >>> >>> On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga wrote: >>> Hi, Attached [1] is the PR for $subject. Please review and merge. [1] https://github.com/wso2/carbon-analytics/pull/156 Thanks, Supun -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 >>> >>> >>> >>> -- >>> *Anjana Fernando* >>> Senior Technical Lead >>> WSO2 Inc. | http://wso2.com >>> lean . enterprise . middleware >>> >> >> >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Merged the PR. Cheers, Anjana. On Wed, Feb 24, 2016 at 1:00 PM, Supun Sethunga wrote: > Hi, > > I have done the changes suggested during the code review. Please review > and merge [1]. > > Will send a separate PR for Compressing the json string. > > > Thanks, > Supun > > On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando wrote: > >> Hi Supun, >> >> Can you please schedule a code review for this. >> >> Cheers, >> Anjana. >> >> On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga wrote: >> >>> Hi, >>> >>> Attached [1] is the PR for $subject. Please review and merge. >>> >>> [1] https://github.com/wso2/carbon-analytics/pull/156 >>> >>> Thanks, >>> Supun >>> >>> -- >>> *Supun Sethunga* >>> Software Engineer >>> WSO2, Inc. >>> http://wso2.com/ >>> lean | enterprise | middleware >>> Mobile : +94 716546324 >>> >> >> >> >> -- >> *Anjana Fernando* >> Senior Technical Lead >> WSO2 Inc. | http://wso2.com >> lean . enterprise . middleware >> > > > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Hi, I have done the changes suggested during the code review. Please review and merge [1]. Will send a separate PR for Compressing the json string. Thanks, Supun On Fri, Feb 19, 2016 at 3:37 PM, Anjana Fernando wrote: > Hi Supun, > > Can you please schedule a code review for this. > > Cheers, > Anjana. > > On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga wrote: > >> Hi, >> >> Attached [1] is the PR for $subject. Please review and merge. >> >> [1] https://github.com/wso2/carbon-analytics/pull/156 >> >> Thanks, >> Supun >> >> -- >> *Supun Sethunga* >> Software Engineer >> WSO2, Inc. >> http://wso2.com/ >> lean | enterprise | middleware >> Mobile : +94 716546324 >> > > > > -- > *Anjana Fernando* > Senior Technical Lead > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] [DAS] Compressed Event Analytics Relation Provider
Hi Supun, Can you please schedule a code review for this. Cheers, Anjana. On Tue, Feb 16, 2016 at 4:30 PM, Supun Sethunga wrote: > Hi, > > Attached [1] is the PR for $subject. Please review and merge. > > [1] https://github.com/wso2/carbon-analytics/pull/156 > > Thanks, > Supun > > -- > *Supun Sethunga* > Software Engineer > WSO2, Inc. > http://wso2.com/ > lean | enterprise | middleware > Mobile : +94 716546324 > -- *Anjana Fernando* Senior Technical Lead WSO2 Inc. | http://wso2.com lean . enterprise . middleware ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
[Dev] [DAS] Compressed Event Analytics Relation Provider
Hi, Attached [1] is the PR for $subject. Please review and merge. [1] https://github.com/wso2/carbon-analytics/pull/156 Thanks, Supun -- *Supun Sethunga* Software Engineer WSO2, Inc. http://wso2.com/ lean | enterprise | middleware Mobile : +94 716546324 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev