nishitpatira opened a new pull request, #4079:
URL: https://github.com/apache/ozone/pull/4079

   ## What changes were proposed in this pull request?
   As part of https://issues.apache.org/jira/browse/HDDS-7372, we are doing 
this in a 2 pronged approach. 
   1. Remove all unused but declared dependencies. This will allow us to 
rationalize the dependencies. 
   2. Add dependencyConvergence for the remaining dependencies to ensure there 
are no missing jars due to version difference across modules. 
   
   This Pull Request is first attempt at 1).
   This allows us to display the work being done to rest of the team and get 
comments. Once there is a consensus on this approach, code changes for other 
modules would be made.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-7372
   
   ## How was this patch tested?
   GitHub Actions. -- 
https://github.com/nishitpatira/ozone/actions/runs/3690850769
   
   Attached are the screenshots of the respective modules `mvn 
dependency:analyze report` 
   <img width="826" alt="Screen Shot 2022-12-13 at 4 41 11 PM" 
src="https://user-images.githubusercontent.com/1568309/207482000-42e05efc-5e87-4a97-89aa-2390da38286d.png";>
   <img width="812" alt="Screen Shot 2022-12-13 at 4 43 08 PM" 
src="https://user-images.githubusercontent.com/1568309/207482004-4063e449-3969-4aea-9bfc-5c6f99cb461b.png";>
   <img width="775" alt="Screen Shot 2022-12-13 at 5 21 37 PM" 
src="https://user-images.githubusercontent.com/1568309/207482013-c0dd3fa7-02fa-468d-9c34-8c9673a0588c.png";>
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to