Re: New message count for Openflow Java in ODL master (Boron)


Michal Polkoráb <michal.polkorab@...>
 

Hi Chinmay,


in that case please go ahead with your changes. You can also remove the two positive counters mentioned before to further improve the performance (we will still have the send messages counters from openflowplugin). Please push one separate commit - where you remove these counters, so that we can track it easily in case the counters are needed in the future.


Regards,

Michal


From: Chinmay Kumar Mohanta <chinmay.kumar.mohanta@...>
Sent: 17 May 2016 08:10
To: Michal Polkoráb; Chinmay.Mohanta
Cc: openflowjava-dev@...; Shuva Jyoti Kar; Ashutosh Bisht; Muthukumaran K
Subject: RE: [openflowjava-dev] New message count for Openflow Java in ODL master (Boron)
 

Hi Michal,

 

From our ongoing effort to debug some of the performance related issues we find that it greatly simplifies the debugging process if error cases (for example like message drop) are reported explicitly rather than implicitly (as a derivation). Explicit error reporting provides  *** sensible & precise contextual*** information which will expedite the process of finding the root cause of the issue.

 

In line with this principle we are still strongly considering to implement this new counter.

 

Pleases comment?

 

Thanks and Regards,

Chinmay Mohanta

 

 

From: Michal Polkoráb [mailto:michal.polkorab@...]
Sent: Friday, May 13, 2016 3:37 PM
To: Chinmay.Mohanta
Cc: openflowjava-dev@...; Shuva Jyoti Kar; Chinmay Kumar Mohanta; Ashutosh Bisht; Muthukumaran K
Subject: Re: [openflowjava-dev] New message count for Openflow Java in ODL master (Boron)

 

Hi Chinmay,

 

your point makes sense. However, we should thoroughly​ consider removing these counters. These were introduced in the past to check if / where there are messages being lost between openflowplugin & openflowjava + to verify the backpressure mechanism. If we similar issue appears again, these counters might be handy.

 

Michal


From: Chinmay.Mohanta <mohanta.chinmay@...>
Sent: 12 May 2016 13:27
To: Michal Polkoráb
Cc: openflowjava-dev@...; shuva.jyoti.kar@...; chinmay.kumar.mohanta@...; ashutosh.bisht@...; muthukumaran.k@...
Subject: Re: [openflowjava-dev] New message count for Openflow Java in ODL master (Boron)

 

Hi Michal,

 

Thanks for your reply.

 

Indeed the formula indicated by you gives the expected value !!

So it obviates the implementation of a new counter.

 

However following new thoughts emerge.

 

Do we really need the counts DS_ENTERED_OFJAVA and DS_ENCODE_SUCCESS ?

 

Maintaining these two counts (for every message) is costly in terms of performance (due to need for synchronization).

 

I think we should have counts only for the error scenarios resulting least influence on the performance.

 

BTW, my goal is to simplify the debugging process to find out whenever/wherever there is a message drop due to resource contention. Message drop count will provide an essential input to rationalize the provisioning of resources. 

 

Thanks and Regards,

Chinmay @ ericsson

 

On Wed, May 11, 2016 at 5:22 PM, Michal Polkoráb <michal.polkorab@...> wrote:

Hello Chinmay,

 

is it really needed to add such counter ? One can get this number by DS_ENTERED_OFJAVA - (DS_ENCODE_SUCCESS + DS_ENCODE_FAIL) if only OutboundQueue is used. This counter looses its meaning when OutboundQueueManager is used as the manager enqueues all messages and the message count is controlled byt rpcQuota in openflowplugin.

 

Regards,

Michal

 


From: Chinmay.Mohanta <mohanta.chinmay@...>
Sent: 10 May 2016 15:22
To: openflowjava-dev@...
Cc: shuva.jyoti.kar@...; chinmay.kumar.mohanta@...; ashutosh.bisht@...; muthukumaran.k@...
Subject: [openflowjava-dev] New message count for Openflow Java in ODL master (Boron)

 

Hi all,

 

To supplement present testing and diagnosing scenarios using message counts, I am proposing following new count.

 

Count name : DS_DROPPED_PACKET_QFULL

Purpose : To count messages dropped because the queue present in ChannelOutboundQueue is full. This count will be helpful to mark the threshold downstream message processing capability of Openflow Java. Downsizing/upsizing resources surrounding this counter will decrease/increase downstream message processing capability.

 

Implementation implications:

 

This new count will be implemented using existing counter module StatisticsCounters.

 

Please share your comments (if any ).

 

Thanks and Regards,

Chinmay @ ericsson

MichalPolkoráb

Software Engineer


Sídlo Mlynské Nivy 56 / 821 05 Bratislava / Slovakia
R&D centrum 
Janka Kráľa 9 /  974 01 Banská Bystrica Slovakia
+421 918 378 907 / michal.polkorab@...
reception: +421 2 206 65 114 / www.pantheon.sk

logo

 

 

MichalPolkoráb

Software Engineer


Sídlo Mlynské Nivy 56 / 821 05 Bratislava / Slovakia
R&D centrum 
Janka Kráľa 9 /  974 01 Banská Bystrica Slovakia
+421 918 378 907 / michal.polkorab@...
reception: +421 2 206 65 114 / www.pantheon.sk

logo

 

MichalPolkoráb

Software Engineer


Sídlo Mlynské Nivy 56 / 821 05 Bratislava / Slovakia
R&D centrum 
Janka Kráľa 9 /  974 01 Banská Bystrica Slovakia
+421 918 378 907 / michal.polkorab@...
reception: +421 2 206 65 114 / www.pantheon.sk

logo

 

Join z.archive.openflowjava-dev@lists.opendaylight.org to automatically receive all group messages.