You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/09/03 15:08:48 UTC

[GitHub] [ozone] dombizita opened a new pull request #2612: HDDS-5695 Review ZK and Curator dependencies, and get rid of them.

dombizita opened a new pull request #2612:
URL: https://github.com/apache/ozone/pull/2612


   ## What changes were proposed in this pull request?
   
   Based on testing, ZK and Curator dependencies aren't neccessary anymore.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-5695
   
   ## How was this patch tested?
   
   Built it without problems.
   


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] fapifta commented on pull request #2612: HDDS-5695 Review ZK and Curator dependencies, and get rid of them.

Posted by GitBox <gi...@apache.org>.
fapifta commented on pull request #2612:
URL: https://github.com/apache/ozone/pull/2612#issuecomment-914459371


   Created https://issues.apache.org/jira/browse/HDDS-5722 to track the two remaining questionable jars.


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] fapifta commented on pull request #2612: HDDS-5695 Review ZK and Curator dependencies, and get rid of them.

Posted by GitBox <gi...@apache.org>.
fapifta commented on pull request #2612:
URL: https://github.com/apache/ozone/pull/2612#issuecomment-912825592


   It seems that some of the jars are not removed from jar-report.txt, although they were added in #2536, @dombizita would you mind take a look and check if there is anything else on the list that we can remove after the tests were removed besides removing the unnecessary ones marked by the dependency check? If there is anything that remains, we have to change the LICENSE.txt in hadoop-ozone/dist/src/main/license/bin as well at the end of the day.


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] fapifta commented on pull request #2612: HDDS-5695 Review ZK and Curator dependencies, and get rid of them.

Posted by GitBox <gi...@apache.org>.
fapifta commented on pull request #2612:
URL: https://github.com/apache/ozone/pull/2612#issuecomment-912807251


   Hi @dombizita thank you for your work on this one. I have merged back the master into HDDS-5447-httpfs branch, and also merged the changes to your PR's branch, in order to have the CI changes and reduce our testing footprint on the branches and PRs whenever it is possible.
   Please don't forget to update your local branches.


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] fapifta merged pull request #2612: HDDS-5695 Review ZK and Curator dependencies, and get rid of them.

Posted by GitBox <gi...@apache.org>.
fapifta merged pull request #2612:
URL: https://github.com/apache/ozone/pull/2612


   


-- 
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: issues-unsubscribe@ozone.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org