Re: [Dev] [APIM] Synchronous call from apim-gateway to Google Analytics

2018-03-08 Thread Chamin Dias
On Wed, Mar 7, 2018 at 8:57 PM, Dushani Wellappili 
wrote:

> Hi all,
>
> please note that I have created an issue [1] in git repository.
>
> [1] https://github.com/wso2/product-apim/issues/2922
>

Also, I think we need to pay attention to this graph

(about
API Latency) when fixing this. It can be used to verify if the time has
reduced.

>
>
>
> *Dushani Wellappili*
> Software Engineer - WSO2
>
> Email : dusha...@wso2.com
> Mobile : +94779367571 <+94%2077%20936%207571>
> Web : https://wso2.com/
>
>
>
> On Wed, Mar 7, 2018 at 5:25 PM, Dushani Wellappili 
> wrote:
>
>> Hi Rukshan,
>>
>> Thank you for the answer. Yes, now in the existing code, event publishing
>> task is done by the same PassThroughMessageProcessor thread.  I will create
>> a git issue.
>>
>>
>> *Dushani Wellappili*
>> Software Engineer - WSO2
>>
>> Email : dusha...@wso2.com
>> Mobile : +94779367571 <+94%2077%20936%207571>
>> Web : https://wso2.com/
>>
>>
>>
>> On Wed, Mar 7, 2018 at 3:34 PM, Rukshan Premathunga 
>> wrote:
>>
>>> Hi Dushani,
>>>
>>> I'm not sure it is possible to publish event async to google APIs. But
>>> we can implement in a way that handover the event publishing task to a
>>> separate thread and continue.
>>> So please create a git issue and we will looking to it.
>>>
>>> Thanks and Regards
>>>
>>> On Wed, Mar 7, 2018 at 3:11 PM, Dushani Wellappili 
>>> wrote:
>>>
 Hi all,

 The api-manager gateway performs a https GET call to publish to google
 analytics [1]. It is a synchronous call.

 If so, the round trip time for an api invocation would depend on the
 Internet connection speed in publishing to google analytics.

 Is there any specific reason for making the call, synchronous?

 Appreciate your comments on this.

 [1] https://github.com/wso2/carbon-commons/blob/master/component
 s/ganalytics/org.wso2.carbon.ganalytics.publisher/src/main/j
 ava/org/wso2/carbon/ganalytics/publisher/GoogleAnalyticsData
 Publisher.java#L81

 Thank you,
 Regards


 *Dushani Wellappili*
 Software Engineer - WSO2

 Email : dusha...@wso2.com
 Mobile : +94779367571 <+94%2077%20936%207571>
 Web : https://wso2.com/



>>>
>>>
>>> --
>>> Rukshan Chathuranga.
>>> Software Engineer.
>>> WSO2, Inc.
>>> +94711822074 <+94%2071%20182%202074>
>>>
>>
>>
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
Chamin Dias
Mobile : 0716097455
Email : cham...@wso2.com
LinkedIn : https://www.linkedin.com/in/chamindias
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [APIM] Synchronous call from apim-gateway to Google Analytics

2018-03-07 Thread Dushani Wellappili
Hi all,

please note that I have created an issue [1] in git repository.

[1] https://github.com/wso2/product-apim/issues/2922


*Dushani Wellappili*
Software Engineer - WSO2

Email : dusha...@wso2.com
Mobile : +94779367571
Web : https://wso2.com/



On Wed, Mar 7, 2018 at 5:25 PM, Dushani Wellappili 
wrote:

> Hi Rukshan,
>
> Thank you for the answer. Yes, now in the existing code, event publishing
> task is done by the same PassThroughMessageProcessor thread.  I will create
> a git issue.
>
>
> *Dushani Wellappili*
> Software Engineer - WSO2
>
> Email : dusha...@wso2.com
> Mobile : +94779367571 <+94%2077%20936%207571>
> Web : https://wso2.com/
>
>
>
> On Wed, Mar 7, 2018 at 3:34 PM, Rukshan Premathunga 
> wrote:
>
>> Hi Dushani,
>>
>> I'm not sure it is possible to publish event async to google APIs. But we
>> can implement in a way that handover the event publishing task to a
>> separate thread and continue.
>> So please create a git issue and we will looking to it.
>>
>> Thanks and Regards
>>
>> On Wed, Mar 7, 2018 at 3:11 PM, Dushani Wellappili 
>> wrote:
>>
>>> Hi all,
>>>
>>> The api-manager gateway performs a https GET call to publish to google
>>> analytics [1]. It is a synchronous call.
>>>
>>> If so, the round trip time for an api invocation would depend on the
>>> Internet connection speed in publishing to google analytics.
>>>
>>> Is there any specific reason for making the call, synchronous?
>>>
>>> Appreciate your comments on this.
>>>
>>> [1] https://github.com/wso2/carbon-commons/blob/master/component
>>> s/ganalytics/org.wso2.carbon.ganalytics.publisher/src/main/j
>>> ava/org/wso2/carbon/ganalytics/publisher/GoogleAnalyticsData
>>> Publisher.java#L81
>>>
>>> Thank you,
>>> Regards
>>>
>>>
>>> *Dushani Wellappili*
>>> Software Engineer - WSO2
>>>
>>> Email : dusha...@wso2.com
>>> Mobile : +94779367571 <+94%2077%20936%207571>
>>> Web : https://wso2.com/
>>>
>>>
>>>
>>
>>
>> --
>> Rukshan Chathuranga.
>> Software Engineer.
>> WSO2, Inc.
>> +94711822074 <+94%2071%20182%202074>
>>
>
>
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [APIM] Synchronous call from apim-gateway to Google Analytics

2018-03-07 Thread Dushani Wellappili
Hi Rukshan,

Thank you for the answer. Yes, now in the existing code, event publishing
task is done by the same PassThroughMessageProcessor thread.  I will create
a git issue.


*Dushani Wellappili*
Software Engineer - WSO2

Email : dusha...@wso2.com
Mobile : +94779367571
Web : https://wso2.com/



On Wed, Mar 7, 2018 at 3:34 PM, Rukshan Premathunga 
wrote:

> Hi Dushani,
>
> I'm not sure it is possible to publish event async to google APIs. But we
> can implement in a way that handover the event publishing task to a
> separate thread and continue.
> So please create a git issue and we will looking to it.
>
> Thanks and Regards
>
> On Wed, Mar 7, 2018 at 3:11 PM, Dushani Wellappili 
> wrote:
>
>> Hi all,
>>
>> The api-manager gateway performs a https GET call to publish to google
>> analytics [1]. It is a synchronous call.
>>
>> If so, the round trip time for an api invocation would depend on the
>> Internet connection speed in publishing to google analytics.
>>
>> Is there any specific reason for making the call, synchronous?
>>
>> Appreciate your comments on this.
>>
>> [1] https://github.com/wso2/carbon-commons/blob/master/component
>> s/ganalytics/org.wso2.carbon.ganalytics.publisher/src/main/
>> java/org/wso2/carbon/ganalytics/publisher/GoogleAna
>> lyticsDataPublisher.java#L81
>>
>> Thank you,
>> Regards
>>
>>
>> *Dushani Wellappili*
>> Software Engineer - WSO2
>>
>> Email : dusha...@wso2.com
>> Mobile : +94779367571 <+94%2077%20936%207571>
>> Web : https://wso2.com/
>>
>>
>>
>
>
> --
> Rukshan Chathuranga.
> Software Engineer.
> WSO2, Inc.
> +94711822074 <+94%2071%20182%202074>
>
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [APIM] Synchronous call from apim-gateway to Google Analytics

2018-03-07 Thread Rukshan Premathunga
Hi Dushani,

I'm not sure it is possible to publish event async to google APIs. But we
can implement in a way that handover the event publishing task to a
separate thread and continue.
So please create a git issue and we will looking to it.

Thanks and Regards

On Wed, Mar 7, 2018 at 3:11 PM, Dushani Wellappili 
wrote:

> Hi all,
>
> The api-manager gateway performs a https GET call to publish to google
> analytics [1]. It is a synchronous call.
>
> If so, the round trip time for an api invocation would depend on the
> Internet connection speed in publishing to google analytics.
>
> Is there any specific reason for making the call, synchronous?
>
> Appreciate your comments on this.
>
> [1] https://github.com/wso2/carbon-commons/blob/master/
> components/ganalytics/org.wso2.carbon.ganalytics.
> publisher/src/main/java/org/wso2/carbon/ganalytics/publisher/
> GoogleAnalyticsDataPublisher.java#L81
>
> Thank you,
> Regards
>
>
> *Dushani Wellappili*
> Software Engineer - WSO2
>
> Email : dusha...@wso2.com
> Mobile : +94779367571 <+94%2077%20936%207571>
> Web : https://wso2.com/
>
>
>


-- 
Rukshan Chathuranga.
Software Engineer.
WSO2, Inc.
+94711822074
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev