rzo1 opened a new pull request, #3572:
URL: https://github.com/apache/storm/pull/3572

   ## What is the purpose of the change
   
   - Removes `com.googlecode.json-simple:json-simple` from Storm core / runtime 
modules  (legacy library 12 years old) and replace it with already included 
json minidev library.
     - This replacement results in some code changes across the project
     - Minidev json is now included in the client shaded deps.
     - Note: `storm-core` clojure integration test relies on `json-simple` 
classes to create a JSON structure and is highly library dependent. As I am not 
speaking clojure good enough, it might be a separate issue to replace the 
library in the test setup, too.
   - Removes `org.jooq:jool` from webui as it was included for the purpose of a 
few methods which can be replaced with Java 11 features.
   - Pins some versions on parent pom to avoid inclusion of the same library in 
the distribution
     - commons-lang3 -> 3.13.0
     - log4j -> 2.20.0
     - slf4j -> 1.7.36
     - Jersey -> 2.40
     - Dropwizard -> 1.3.29
     - j2html -> 1.6.0
     - minidev json -> 2.5.0
     - JUnit 5.10.0
   - Excludes some garbage from autocreds as jdbc/mssql driver is not needed 
for runtime operations
   
   If this reviewed, I will create Jira issues for every version upgrade 
included in this PR in order to have them in the next release logs. 
   
   ## How was the change tested
   
   - GH actions


-- 
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: dev-unsubscr...@storm.apache.org

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

Reply via email to