[GitHub] [incubator-linkis] casionone commented on pull request #2116: linkis-storage module replace cglib with Use spring inline cglib

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

casionone commented on PR #2116:
URL: https://github.com/apache/incubator-linkis/pull/2116#issuecomment-1127700359

   LGTM 
   According to https://stackoverflow.com/questions/41478307/whats-the-difference-between-spring-cglib-and-cglib,
   we can know :
   
   Cglib was inlined into Spring as of version 3.2.0 as it is mentioned in the [release notes of this version](http://docs.spring.io/spring/docs/3.2.0.RELEASE/spring-framework-reference/html/migration-3.2.html):
   
   In prior versions, users of Spring's subclass-based AOP proxies (e.g. via proxy-target-class="true") and @Configuration class support were required to declare an explicit dependency on CGLIB 2.2. As of Spring Framework 3.2, we now repackage and inline the newly-released CGLIB 3.0.
   
   This means greater convenience for users, as well as correct functionality for Java 7 users who are creating subclass proxies of types that contain invokedynamic bytecode instructions. Repackaging CGLIB internally ensures no classpath conflicts with other third party frameworks that may depend on other versions of CGLIB.
   
   This was done to provide automatic updates that correlate with cglib and avoiding of version conflicts as cglib somestimes breaks its API.
   


-- 
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@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org