You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/01/25 23:39:45 UTC

[GitHub] [bookkeeper] dlg99 opened a new pull request #3010: Forced the same version of guava (plus upgraded it); fixed deprecations etc.

dlg99 opened a new pull request #3010:
URL: https://github.com/apache/bookkeeper/pull/3010


   ### Motivation
   
   Older versions of guava (w/CVEs) used in some subprojects
   
   ### Changes
   
   Forced the same version of guava; fixed deprecation problems (murmur3_32) and compilation problems (checkArgument).
   checkArgument cannot be statically imported because there are now overrides of it; checkstyle was not very cooperative so I had to remove the import altogether. 
   


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

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



[GitHub] [bookkeeper] eolivelli commented on pull request #3010: Forced the same version of guava (plus upgraded it); fixed deprecations etc.

Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #3010:
URL: https://github.com/apache/bookkeeper/pull/3010#issuecomment-1023217089


   please fix the conflicts


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

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



[GitHub] [bookkeeper] dlg99 merged pull request #3010: Forced the same version of guava (plus upgraded it); fixed deprecations etc.

Posted by GitBox <gi...@apache.org>.
dlg99 merged pull request #3010:
URL: https://github.com/apache/bookkeeper/pull/3010


   


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

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



[GitHub] [bookkeeper] dlg99 commented on pull request #3010: Forced the same version of guava (plus upgraded it); fixed deprecations etc.

Posted by GitBox <gi...@apache.org>.
dlg99 commented on pull request #3010:
URL: https://github.com/apache/bookkeeper/pull/3010#issuecomment-1023501648


   @eolivelli I rebased it, I believe we'll get conflicts in either this or netty update PR, depends on which one is merged first.


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

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