[GitHub] bookkeeper issue #217: Code cleanups

2017-06-30 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/217 The change looks good, Please open an issue or a Jira ticket for this PR, and at the end of this PR description, For the checks that you have done, please place "x" in the "[ ]&

[GitHub] bookkeeper issue #198: TestBackwardCompat.testCompat410 often fails due to i...

2017-06-29 Thread jiazhai
Github user jiazhai closed the issue at: https://github.com/apache/bookkeeper/issues/198 --- 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

[GitHub] bookkeeper pull request #219: Issue 198: TestBackwardCompat.testCompat410 of...

2017-06-29 Thread jiazhai
Github user jiazhai closed the pull request at: https://github.com/apache/bookkeeper/pull/219 --- 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

[GitHub] bookkeeper issue #205: Issue 208: Improve ledger fence logic

2017-06-28 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/205 Seems there is conflict files. --- 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

[GitHub] bookkeeper pull request #213: Issue 212: change github PR template to adjust...

2017-06-25 Thread jiazhai
GitHub user jiazhai opened a pull request: https://github.com/apache/bookkeeper/pull/213 Issue 212: change github PR template to adjust both issue# and jira# Descriptions of the changes in this PR: Change PR template to adjust both issue# and jira# --- Be sure to do

[GitHub] bookkeeper issue #212: Change pull request template to adjust both issue# an...

2017-06-25 Thread jiazhai
GitHub user jiazhai opened an issue: https://github.com/apache/bookkeeper/issues/212 Change pull request template to adjust both issue# and jira# **BUG REPORT** Please describe the issue you observed: - What did you do? create a pull request for JIRA ticket

[GitHub] bookkeeper issue #211: change bk-merge script to delete the last part of PR ...

2017-06-25 Thread jiazhai
GitHub user jiazhai opened an issue: https://github.com/apache/bookkeeper/issues/211 change bk-merge script to delete the last part of PR description. 1. Please describe the feature you are requesting. Change bk-merge script to delete the last part of PR description. When

[GitHub] bookkeeper issue #210: BOOKKEEPER-1100: Add Http Server for Bookkeeper

2017-06-25 Thread jiazhai
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 f

[GitHub] bookkeeper issue #200: BookKeeper Client: Speculative Read/Read LAC policy

2017-06-22 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/200 @eolivelli, thanks for the info. @sijie, excuse me, there maybe some mis-sync regarding this. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] bookkeeper issue #199: Change the HashSet to BitSet to handle duplicated boo...

2017-06-21 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/199 Would you please open an Issue for it, and change the name of this PR as "Issue XXX: Description " ? --- If your project is set up for it, you can reply to this email and have

[GitHub] bookkeeper issue #200: BookKeeper Client: Speculative Read/Read LAC policy

2017-06-21 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/200 Would you please open an Issue for it, and change the name of this PR as "Issue XXX: Description " ? --- If your project is set up for it, you can reply to this email and have

[GitHub] bookkeeper issue #197: BOOKKEEPER-974: Add an official bookkeeper docker ima...

2017-06-21 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/197 There seems be an empty file: "docker/4.4.0/.keep", what is the reason to have it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] bookkeeper issue #194: Issue 184: Support GitHub issues in bk-merge script

2017-06-19 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/194 cool --- 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

[GitHub] bookkeeper issue #192: Issue 191: add template for ISSUE and PR

2017-06-15 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/192 @sijie Done the fix, excluded .github dir. Thanks a lot. --- 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

[GitHub] bookkeeper pull request #192: Issue 191: add template for ISSUE and PR

2017-06-15 Thread jiazhai
GitHub user jiazhai opened a pull request: https://github.com/apache/bookkeeper/pull/192 Issue 191: add template for ISSUE and PR Following BP-9, provide a template for ISSUE and PR. When create issue, will use ISSUE template; while create PR will use PR template. You can

[GitHub] bookkeeper issue #191: Github issue and pull request template

2017-06-15 Thread jiazhai
GitHub user jiazhai opened an issue: https://github.com/apache/bookkeeper/issues/191 Github issue and pull request template Regarding use "github issue" feature, templates for issue and pull request is needed. would like to use this issue to create one for them. [H

[GitHub] bookkeeper issue #174: BOOKKEEPER-748: Move fence requests out of read threa...

2017-06-01 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/174 This is a great feature. --- 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

[GitHub] bookkeeper issue #169: BOOKKEEPER-1081: a docker file for Apache BookKeeper

2017-05-28 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/169 Since there is a dup related work, will close this PR first, discuss and collaborate together with @eolivelli and @caiok to make it done. --- If your project is set up for it, you can reply

[GitHub] bookkeeper pull request #169: BOOKKEEPER-1081: a docker file for Apache Book...

2017-05-28 Thread jiazhai
Github user jiazhai closed the pull request at: https://github.com/apache/bookkeeper/pull/169 --- 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

[GitHub] bookkeeper pull request #169: BOOKKEEPER-1081: a docker file for Apache Book...

2017-05-27 Thread jiazhai
GitHub user jiazhai opened a pull request: https://github.com/apache/bookkeeper/pull/169 BOOKKEEPER-1081: a docker file for Apache BookKeeper A docker file for Apache BookKeeper. This docker image will start a new bookie instance. Steps to run it: 1, start zookeeper

[GitHub] bookkeeper issue #138: BOOKKEEPER-1008: Netty 4.1

2017-05-11 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/138 Passed all the test on my Mac in 17 min. --- 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

[GitHub] bookkeeper issue #80: BOOKKEEPER-816: use native fallocate to improve journa...

2017-05-03 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/80 Oh. Sorry. I did not use Pnative. --- 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

[GitHub] bookkeeper issue #80: BOOKKEEPER-816: use native fallocate to improve journa...

2017-05-03 Thread jiazhai
Github user jiazhai commented on the issue: https://github.com/apache/bookkeeper/pull/80 build success on ubuntu 16.04. ``` [INFO] bookkeeper . SUCCESS [ 0.836 s] [INFO] compability dependencies ... SUCCESS