[GitHub] bettermanzzy opened a new issue #1057: Upgrading Zipkin 2.4.2 to Zipkin 2.11.12, should we remain the earlier version suporting?

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

bettermanzzy opened a new issue #1057: Upgrading Zipkin 2.4.2 to Zipkin 2.11.12, should we remain the earlier version suporting?
URL: https://github.com/apache/servicecomb-java-chassis/issues/1057
 
 
   Now, I am going to upgrade the version of zipkin supporting in java-chassis.
   
   Version of the zipkin kit supported in Java-Chassis now,
   Brave 4.13.1, zipkin 2.4.2, zipkin-reporter 2.2.2. 
   Version of the zipkin kit that I am going to upgrade to,
   Brave 5.6.0, zipkin 2.11.12, zipkin.java 2.10.1, zipkin-reporter 2.7.13.  
   
   In the new zipkin kit , there are some APIs have been deprecated, such as, MDCCurrentTraceContext, StrictCurrentTraceContext, etc. Therefore, if I upgrade it, the Java-Chassis could not support the earlier one. Should I remain the related code of earlier zipkin in Java-Chassis?
   
   To my opinion, I just have to upgrade it through replace the earlier zipkin kit with new one. In that way, the Java-Chassis can only support the zipkin 2.11.12 but not zipkin 2.4.2.
   
   What about other's opinion?please feel free to let me know.

----------------------------------------------------------------
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