[GitHub] ddragosd commented on issue #2282: Distributed tracing support #2192

[gi...@git.apache.org]

ddragosd commented on issue #2282: Distributed tracing support #2192
URL: https://github.com/apache/incubator-openwhisk/pull/2282#issuecomment-308511536
 
 
   I'm echoing Tyson's suggestion to go ahead and merge. Myself, my team for instance, are currently blocked by this. We need this information in order to understand performance bottlenecks, and make performance suggestions and improvements. We also need a measurable and reproducible way to see the impact of any change we make in all our deployments and environments.   
   
   Pragmatically speaking if we want pluggability, then we're blocking this PR for an unknown amount of time.
    
   For now, I think that, if we could agree on items 2. and 3. that you mentioned, which seem like a good way to manage 1. (there's a flag to turn tracing `off`) , then we might have something good enough to make progress.  We can always refactor it later, after the pluggability feature is in.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services