Github user sijie commented on the issue:
https://github.com/apache/bookkeeper/pull/210
@jvrao do you mind sending out your proposal. we might need to merge the
efforts here, and also @eolivelli has some ideas on how to implement this with
standard servlets.
---
If your project is
Github user jvrao commented on the issue:
https://github.com/apache/bookkeeper/pull/210
We are adding REST endpoint to BookKeeper. I need to check if they overlap.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user sijie commented on the issue:
https://github.com/apache/bookkeeper/pull/210
regarding the servlet, I agree with @eolivelli , we probably should define
a standard API and allow it running with different web container. @yzang : do
finagle and vertx support servlet?
---
If
Github user eolivelli commented on the issue:
https://github.com/apache/bookkeeper/pull/210
Some comments:
1) is it possible NOT to bundle all the implementations in
bookkeeper-server project ?
Or at least mark the dependencies as "provided" o "runtime" so that we do
not
Github user yzang commented on the issue:
https://github.com/apache/bookkeeper/pull/210
Found a few flaky/failing tests not related to this commit:
Running org.apache.bookkeeper.meta.TestLongZkLedgerIdGenerator
Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time
Github user jiazhai commented on the issue:
https://github.com/apache/bookkeeper/pull/210
@yzang, Would you please help change the last part of this PR description a
little? If you did the checks, put an "x" in the "[ ]".
---
If your project is set up for it, you can reply to this