mrproliu commented on pull request #8193: URL: https://github.com/apache/skywalking/pull/8193#issuecomment-980756236
> For the original protocol it only adds a Boolean check that’s ok, what I meant is the new protocol for Satellite, it parses the node identifier every time, I just don’t want to move the performance cost from Satellite to OAP Ok, understand. But if the satellite also using the original protocol, the OAP also only support one identify for one streaming, the satellite will open, send and close the stream every time. But we can cross the identity in one stream, I think it good for they both, because we reduce the connect and close time. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
