[jira] [Commented] (BOOKKEEPER-1033) Handle DirsPartitionDuplication

2017-06-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1033?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16051035#comment-16051035 ] ASF GitHub Bot commented on BOOKKEEPER-1033: Github user eolivelli commen

[GitHub] bookkeeper issue #189: BOOKKEEPER-1033: Handle DirsPartitionDuplication

2017-06-15 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/189 @sijie as far as I can see the flag only adds an additional check on the configuration and it does not affect the way we are going to calculate disk space (which now it will be correct).

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

2017-06-15 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/194 +1 I tested it and it is working fine. Awesome ! --- 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 hav

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

2017-06-15 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/192 +1 I will try the new script at #194 againts this pr --- 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 h

[GitHub] bookkeeper issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper

2017-06-15 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/bookkeeper/pull/183 Overall is ok, but rat is failing and on jenkins one of the new tests failed, the mixed cluster one Can you fix it? --- If your project is set up for it, you can reply to this em

[jira] [Commented] (BOOKKEEPER-588) SSL support

2017-06-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16051005#comment-16051005 ] ASF GitHub Bot commented on BOOKKEEPER-588: --- Github user eolivelli commented

[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 n

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

2017-06-15 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/bookkeeper/pull/194 The script change for #184 . @eolivelli --- 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 e

[GitHub] bookkeeper pull request #194: Issue 184: Support GitHub issues in bk-merge s...

2017-06-15 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/bookkeeper/pull/194 Issue 184: Support GitHub issues in bk-merge script - Recoganize the issue title and normalize it - Close the corresponding issue when closing the pr You can merge this pull request into a Git

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

2017-06-15 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/bookkeeper/pull/192 @jiazhai I think these two files introduced the rate-check warnings. please either exclude them from apache-rat checking or add apache license header. --- If your project is set up for it, you can

Re: [VOTE] BP-9: Github issues for Issue Tracking

2017-06-15 Thread Enrico Olivelli
+1 Il gio 15 giu 2017, 19:52 Sijie Guo ha scritto: > +1 from me. > > As we discussed today morning, Enrico raised the concern about merging > directly from Github. The conclusion that we achieved: > > - At this point, we will still use the merge script for merging pull > requests. The merge scri

Re: BP-9 - Github issues for Issue Tracking

2017-06-15 Thread Jia Zhai
FYI. created bookkeeper [issue]( https://github.com/apache/bookkeeper/issues/191) and [PR]( https://github.com/apache/bookkeeper/pull/192), We could try them out to see what permissions we have now under Apache’s mirror github. On Mon, Jun 12, 2017 at 2:50 PM, Jia Zhai wrote: > Thanks Sijie and

Re: [VOTE] BP-9: Github issues for Issue Tracking

2017-06-15 Thread Sijie Guo
+1 from me. As we discussed today morning, Enrico raised the concern about merging directly from Github. The conclusion that we achieved: - At this point, we will still use the merge script for merging pull requests. The merge script will be changed to handle Github Issues. - At the same time, Ji

[jira] [Commented] (BOOKKEEPER-1086) Ledger Recovery - Refactor PendingReadOp

2017-06-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16050856#comment-16050856 ] ASF GitHub Bot commented on BOOKKEEPER-1086: GitHub user athanatos opened

[GitHub] bookkeeper pull request #193: BOOKKEEPER-1086: ZkUnderreplicationManager cac...

2017-06-15 Thread athanatos
GitHub user athanatos opened a pull request: https://github.com/apache/bookkeeper/pull/193 BOOKKEEPER-1086: ZkUnderreplicationManager cache watcher Previously, getLedgerToReplicate left watches each time it traversed the tree until it found a suitable replication target. Since w

Re: [VOTE] Merge DistributedLog as the subproject of Apache BookKeeper

2017-06-15 Thread Sijie Guo
/cc (dev@bookkeeper.apache.org and d...@pulsar.incubator.apache.org) Closed the vote. The vote passed with 10 +1 and no -1. The 10 +1 includes 3 +1 (binding) and 7 +1 (non-binding) 3 +1 (binding): - Franck Cuny - Leigh Stewart - Sijie Guo 7 +1 (non-binding) - Edward Ribeiro - Gerrit Sundaram

[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 merge

[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. [Here](https:/

Re: Draft Agenda for Bi-weekly meeting at 2017-06-15

2017-06-15 Thread Enrico Olivelli
Great! Thank you Jia Enrico Il gio 15 giu 2017, 18:56 Jia Zhai ha scritto: > Updated the meeting minutes > < > https://cwiki.apache.org/confluence/display/BOOKKEEPER/2017-06-15+Meeting+notes > >, > if there is anything missing, please help add it. > Thanks a lot. > > On Tue, Jun 13, 2017 at 1:3

Re: Draft Agenda for Bi-weekly meeting at 2017-06-15

2017-06-15 Thread Jia Zhai
Updated the meeting minutes , if there is anything missing, please help add it. Thanks a lot. On Tue, Jun 13, 2017 at 1:39 PM, Jia Zhai wrote: > Hi all, :) > I would like to put a draft Agenda here >

Build failed in Jenkins: bookkeeper-master #1785

2017-06-15 Thread Apache Jenkins Server
See -- [...truncated 1.11 MB...] Tests run: 80, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 56.684 sec - in org.apache.bookkeeper.test.BookieFailureTest Running org.apache.bookkee