You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by "gaozhangmin (via GitHub)" <gi...@apache.org> on 2023/03/23 03:23:40 UTC

[GitHub] [bookkeeper] gaozhangmin opened a new pull request, #3881: Fix cannot transit to writable mode from forceReadOnly mode

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

   ## Motivation
   
   When running in a "forceReadOnly" mode (by starting Bookie with the "readonly" parameter), attempts to invoke the API `PUT /api/v1/bookie/state/readonly` to transition to a writable mode results in no feedback or messages indicating that the mode has not been updated successfully.
   
   ## Changes
   
   When attempting to transition from "forceReadOnly" mode to a writable mode, a BAD_REQUEST error will be outputted stating that the transition cannot be completed because the server is currently in "forceReadOnly" mode. This will provide more meaningful feedback to users attempting to transition out of this mode.


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

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

   rerun failure checks


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

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

   rerun failure checks


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

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

   rerun failure checks


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


[GitHub] [bookkeeper] hangc0276 commented on pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

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

   LGTM


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


[GitHub] [bookkeeper] gaozhangmin closed pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

Posted by "gaozhangmin (via GitHub)" <gi...@apache.org>.
gaozhangmin closed pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode
URL: https://github.com/apache/bookkeeper/pull/3881


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

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

   rerun failure checks


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

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

   rerun failure checks


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


[GitHub] [bookkeeper] nicoloboschi merged pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

Posted by "nicoloboschi (via GitHub)" <gi...@apache.org>.
nicoloboschi merged PR #3881:
URL: https://github.com/apache/bookkeeper/pull/3881


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

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

   rerun failure checks


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

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

   rerun failure checks


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


[GitHub] [bookkeeper] gaozhangmin closed pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode

Posted by "gaozhangmin (via GitHub)" <gi...@apache.org>.
gaozhangmin closed pull request #3881: Fix cannot transit to writable mode from forceReadOnly mode
URL: https://github.com/apache/bookkeeper/pull/3881


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


[GitHub] [bookkeeper] gaozhangmin commented on pull request #3881: [Bookie] Prevent transit to writable mode when forceReadOnly mode is active

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

   @eolivelli  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