Issue-553 Documentation for new API (WIP)#728
Issue-553 Documentation for new API (WIP)#728eolivelli wants to merge 1 commit intoapache:masterfrom
Conversation
|
@sijie can you take a first look ? |
|
Seems #727 is going on, could we merge this after that, and change some of the description according to it? |
|
FAILURE --none-- |
|
The API documentation should be the final change after we sort out all the api related issues. |
|
|
||
| ## New API | ||
|
|
||
| Since 4.6 BookKeeper provides a new client API which leverages Java CompletableFuture facility. |
There was a problem hiding this comment.
you need also document
- delete op
- append api in WriteHandle
- read api in ReadHandle
Also you need to update ledgeradv api page as well.
|
@eolivelli can you help pick this up? This will be blocking 4.6.0. |
|
Yes. I stopped for a while. I will address comments today |
|
@eolivelli this issue has higher priority than BP18. would appreciate you help completed this. if you don't have time, I am okay to take it. |
|
I am doing this today (for me is early morning now), I will update the pr soon. I know this is a priority. I think you are faster for many reasons. |
|
@eolivelli it is not clear to me. are you picking this up, or not? regarding other blockers, we are trying to clean them up this week. |
|
Please pick this up, I think it is better. |
|
@eolivelli I sent the pull request #764 . I will close this one since there is no activity on this one anymore. |
4 participants
Add documentation for new API