You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by "shoothzj (via GitHub)" <gi...@apache.org> on 2024/05/16 11:37:18 UTC

[PR] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

shoothzj opened a new pull request, #4368:
URL: https://github.com/apache/bookkeeper/pull/4368

   ### Motivation
   
   Bump jetcd from 0.7.7 to 0.8.0 to more compatible with grpc 1.64.0/protobuf 3.25.1. See also #3849 #4344


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

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


Re: [PR] [WIP] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj commented on PR #4368:
URL: https://github.com/apache/bookkeeper/pull/4368#issuecomment-2131236475

   @lhotari BookKeeper's client and server in one module and the fat client makes thing complicated. I would like to separate client and server module first. But the client module will still depends on etcd. I think maybe we need go to jdk11 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: commits-unsubscribe@bookkeeper.apache.org

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


Re: [PR] [WIP] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj commented on PR #4368:
URL: https://github.com/apache/bookkeeper/pull/4368#issuecomment-2115026985

   @lhotari jetcd 0.8.0 required jdk11, I think we need to do more jobs. I would convert this pr to draft 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: commits-unsubscribe@bookkeeper.apache.org

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


Re: [PR] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj commented on PR #4368:
URL: https://github.com/apache/bookkeeper/pull/4368#issuecomment-2115001393

   @lhotari PTAL


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

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


Re: [PR] [WIP] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj closed pull request #4368: [WIP] Bump jetcd from 0.7.7 to 0.8.0
URL: https://github.com/apache/bookkeeper/pull/4368


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

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


Re: [PR] [WIP] Bump jetcd from 0.7.7 to 0.8.0 [bookkeeper]

Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #4368:
URL: https://github.com/apache/bookkeeper/pull/4368#issuecomment-2115078748

   > @lhotari jetcd 0.8.0 required jdk11, I think we need to do more jobs. I would convert this pr to draft first
   
   Do we have plans to bump minimum jdk version requirement in Bookkeeper? I'd prefer JDK 17.


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

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