bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

Published 20 hours ago • verified publisher icon apache

Add force readonly param to RESTful API

Open labuladong opened this issue 2 years ago • 3 comments

Motivation

Allow readonly API to force bookie readonly.

labuladong avatar Nov 10 '23 03:11 labuladong 

Error:  Errors: 
Error:  org.apache.bookkeeper.server.http.TestHttpService.testBookieReadOnlyState
Error:    Run 1: TestHttpService.testBookieReadOnlyState:983 » NullPointer
Error:    Run 2: TestHttpService.testBookieReadOnlyState:983 » NullPointer
Error:    Run 3: TestHttpService.testBookieReadOnlyState:983 » NullPointer

Seems related to this change

zymap avatar Nov 14 '23 03:11 zymap

The original API already supports setting read-only. Why is the force parameter added here?

dragonls avatar Nov 23 '23 07:11 dragonls

@labuladong Please rebase the master, thanks.

hangc0276 avatar Dec 07 '23 04:12 hangc0276