zhaoyuguang commented on pull request #7190: URL: https://github.com/apache/skywalking/pull/7190#issuecomment-870211112
> > @liqiangz Sentinel has a large number of users. With the introduction of Sentinel, the application service will generate a large number of LocalSpan as currently designed. From this point of view, is it better to put it in an optional plug-in? > > Because the similar hystrix plugin is now placed in the default, so I put it in the default plugin. But what you said also makes sense, it better be put in optional plugins. Yes, the Hystrix Plugin does produce a lot of LocalSpans, but it's a little different than Sentinel - Not import by the Hystrix Plugin will result in trace chain breakage, which Sentinel will not breakage, because Hystrix mostly implements isolation for thread pools - Hystrix requires secondary development by developers, but Sentinel is compatible with almost every framework, developers just import Sentinel in pom.xml -- 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]
