GitHub user yzang opened a pull request:
https://github.com/apache/bookkeeper/pull/210
BOOKKEEPER-1100: Add Http Server for Bookkeeper
BOOKKEEPER-1100:
Provide a general interface for HttpServer, which can be easily implemented
by different HTTP frameworks.
This change include two implementations of HttpServer, one is TwitterServer
and another one is Vertx.
Provide a general AbstractHandlerFactory, which is able to create handlers
to handler different http apis,
and is able to be implemented for different HTTP frameworks.
Provide a service level classes, which include all the logics that http
handlers will use.
Add test case for HttpServer
Descriptions of the changes in this PR:
xxxxxx
---
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:
- [ ] Make sure the PR title is formatted like:
`<Issue #>: Description of pull request`
`e.g. Issue 123: Description ...`
- [ ] Make sure tests pass via `mvn clean apache-rat:check install
findbugs:check`.
- [ ] Replace `<Issue #>` in the title with the actual Issue number, if
there is one.
---
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yzang/bookkeeper
yzang/http/yzang/BOOKKEEPER-1100
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/bookkeeper/pull/210.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #210
----
commit 4ba1bf02b43a8cbe6357ee3d9e655ffa76351571
Author: Yiming Zang <[email protected]>
Date: 2017-06-25T03:29:52Z
Add Http Server for Bookkeeper.
Provide a general interface for HttpServer, which can be easily implemented
by different HTTP frameworks.
This change include two implementations of HttpServer, one is TwitterServer
and another one is Vertx.
Provide a general AbstractHandlerFactory, which is able to create handlers
to handler different http apis,
and is able to be implemented for different HTTP frameworks.
Provide a service level classes, which include all the logics that http
handlers will use.
Add test case for HttpServer
----
---
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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---