[GitHub] [skywalking] kezhenxu94 edited a comment on pull request #8193: Introduce the Customized ALS Protocol for Satellite

[GitBox <gi...@apache.org>]

kezhenxu94 edited a comment on pull request #8193:
URL: https://github.com/apache/skywalking/pull/8193#issuecomment-980809798


   Just 
   
   > Let's back to the original question, where shows OAP will increase load? Why do you say there is a performance impact to OAP? @mrproliu
   > From my understanding of this solution, there is only another new grpc service introduced, no performanceimpact for OAP at all.
   
   The problem is that the new protocol is to replace the old one essentially, think in this way, an ALS message will be processed by and only by one of the protocols, if it’s processed by the old one (before this new Satellite protocol) it is (highly) possible that we don’t need to parse the node identifier but if it’s processed by the new one it always needs to parse the node identifier, do you agree the parsing of node identifier is a waste of resources?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org