ivankelly commented on a change in pull request #521: Issue 520: Add more http endpoint URL: https://github.com/apache/bookkeeper/pull/521#discussion_r141305625
########## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/HttpRouter.java ########## @@ -33,14 +33,14 @@ public static final String HEARTBEAT = "/heartbeat"; public static final String SERVER_CONFIG = "/api/v1/config/server_config"; // bookkeeper - public static final String LIST_BOOKIES = "/api/v1/bookkeeper/list_bookies"; - public static final String UPDATE_COOKIE = "/api/v1/bookkeeper/update_cookie"; + // ledger public static final String DELETE_LEDGER = "/api/v1/ledger/delete"; public static final String LIST_LEDGER = "/api/v1/ledger/list"; public static final String GET_LEDGER_META = "/api/v1/ledger/metadata"; public static final String READ_LEDGER_ENTRY = "/api/v1/ledger/read"; // bookie + public static final String LIST_BOOKIES = "/api/v1/bookie/list_bookies"; Review comment: This isn't a resource of the bookie, but rather a resource of the cluster ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
