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

2017-06-12 Thread Leigh Stewart
+1 On Fri, Jun 9, 2017 at 2:48 PM, Uma gangumalla wrote: > +1 (binding) > > Regards, > Uma > > On Thu, Jun 8, 2017 at 5:21 PM, Sijie Guo wrote: > > > ( /cc bookkeeper dev@ and incubator general@ for awareness ) > > > > Hi all, > > > > There was a joint discussion between BookKeeper PMC and Dist

[25/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-client/src/test/java/org/apache/distributedlog/client/routing/TestConsistentHashRoutingService.java -- diff --git a/distributedlog-client

[09/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/bookkeeper/stats/ServletReporter.java -- diff --git a/distributedlog-service/src/main/java/org/apache/bo

[30/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
DL-205: Remove StatusCode dependency on DLException - Remove StatusCode from exceptions. Use integer as exception codes. - Also re-organize the modules: - [ ] distributedlog-protocol (for core structures) and distributedlog-core (for core library). - [ ] proxy: distributedlog-proxy-protocol (new

[04/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/distributedlog/service/stream/admin/StreamAdminOp.java -- diff --git a/distributedlog-service/src/main/j

[11/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/test/java/org/apache/distributedlog/service/TestRegionUnavailable.java -- diff --git a/distributedlog-proxy-server/src/t

[14/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/stream/StreamImpl.java -- diff --git a/distributedlog-proxy-server/src/main/

[13/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/stream/admin/StreamAdminOp.java -- diff --git a/distributedlog-proxy-server/

[15/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/placement/ZKPlacementStateManager.java -- diff --git a/distributedlog-proxy-

[22/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-client/src/main/java/org/apache/distributedlog/client/routing/ConsistentHashRoutingService.java -- diff --git a/distributedlog-prox

[19/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-protocol/pom.xml -- diff --git a/distributedlog-proxy-protocol/pom.xml b/distributedlog-proxy-protocol/pom.xml new file mode 100644

[29/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-client/src/main/java/org/apache/distributedlog/client/DistributedLogClientImpl.java -- diff --git a/distributedlog-client/src/main/java/o

[18/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/ClientUtils.java -- diff --git a/distributedlog-proxy-server/src/main/java/o

[08/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/distributedlog/service/MonitorService.java -- diff --git a/distributedlog-service/src/main/java/org/apac

[21/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-client/src/main/java/org/apache/distributedlog/client/speculative/DefaultSpeculativeRequestExecutionPolicy.java -- diff --git a/dis

[16/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/balancer/SimpleBalancer.java -- diff --git a/distributedlog-proxy-server/src

[06/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/distributedlog/service/placement/ZKPlacementStateManager.java -- diff --git a/distributedlog-service/src

[17/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/main/java/org/apache/distributedlog/service/MonitorService.java -- diff --git a/distributedlog-proxy-server/src/main/jav

[28/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-client/src/main/java/org/apache/distributedlog/client/proxy/ProxyClient.java -- diff --git a/distributedlog-client/src/main/java/org/apac

[20/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-client/src/test/java/org/apache/distributedlog/client/ownership/TestOwnershipCache.java -- diff --git a/distributedlog-proxy-client

[10/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/test/java/org/apache/distributedlog/service/stream/limiter/TestServiceRequestLimiter.java -- diff --git a/distributedlog

[02/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/test/java/org/apache/distributedlog/service/TestRegionUnavailable.java -- diff --git a/distributedlog-service/src/test/java/o

[12/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-server/src/test/java/org/apache/distributedlog/service/DistributedLogServerTestCase.java -- diff --git a/distributedlog-proxy-serve

[01/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 52c0eef87 -> c44e0278e http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/test/java/org/apache/distributedlog/service/stream/limiter/TestServiceRequestLimiter.java

[05/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/distributedlog/service/stream/StreamImpl.java -- diff --git a/distributedlog-service/src/main/java/org/a

[03/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/test/java/org/apache/distributedlog/service/DistributedLogServerTestCase.java -- diff --git a/distributedlog-service/src/test

[23/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-proxy-client/src/main/java/org/apache/distributedlog/client/DistributedLogMultiStreamWriter.java -- diff --git a/distributedlog-proxy-cli

[24/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-protocol/src/main/java/org/apache/distributedlog/exceptions/TransactionIdOutOfOrderException.java -- diff --git a/distributedlog-protocol

[26/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-client/src/main/java/org/apache/distributedlog/service/DistributedLogClientBuilder.java -- diff --git a/distributedlog-client/src/main/ja

[07/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-service/src/main/java/org/apache/distributedlog/service/balancer/SimpleBalancer.java -- diff --git a/distributedlog-service/src/main/java

[27/30] incubator-distributedlog git commit: DL-205: Remove StatusCode dependency on DLException

2017-06-12 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/c44e0278/distributedlog-client/src/main/java/org/apache/distributedlog/client/routing/ServerSetRoutingService.java -- diff --git a/distributedlog-client/src/main

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

2017-06-12 Thread Franck Cuny
+1 for subproject. On Mon, Jun 12, 2017 at 7:57 AM Leigh Stewart wrote: > +1 > > On Fri, Jun 9, 2017 at 2:48 PM, Uma gangumalla > wrote: > > > +1 (binding) > > > > Regards, > > Uma > > > > On Thu, Jun 8, 2017 at 5:21 PM, Sijie Guo wrote: > > > > > ( /cc bookkeeper dev@ and incubator general@ f

Github repo isn't in-sync with the git repo

2017-06-12 Thread Sijie Guo
It seems that the recent merged changes are not propagated into the github mirror. it seems something is broken in INFRA. try to asking INFRA team.

[GitHub] incubator-distributedlog pull request #131: DL-205: Remove StatusCode depend...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/131 --- 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 t

[GitHub] incubator-distributedlog pull request #130: DL-199: Be able to support files...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/130 --- 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 t

incubator-distributedlog git commit: DL-199: Be able to support filesystem-path like name

2017-06-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master c44e0278e -> 9c6c9c452 DL-199: Be able to support filesystem-path like name In order to support hierarchical namespace, we need to be able to support filesystem path like log name. Author: Sijie Guo Reviewers: Jia Zha

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

2017-06-12 Thread Dave Fisher
Hi - I am adding the Pulsar Dev list so that podling can be certainly aware. (I know that there are BookKeeper dev involved in Pulsar. Please drop general@ off for any further discussions from the Pulsar side. Regards, Dave > On Jun 8, 2017, at 5:21 PM, Sijie Guo wrote: > > ( /cc bookkeeper

[2/2] incubator-distributedlog git commit: DL-204: Bump libthrift to latest version for distributedlog-core

2017-06-12 Thread sijie
DL-204: Bump libthrift to latest version for distributedlog-core Currently finagle heavily depends on an out-of-dated version - libthrift 5.0. Proxy modules (client, server) depend on this version, however the core library doesn't really depend on libthrift. This change is to change libthrift t

[GitHub] incubator-distributedlog pull request #132: DL-204: Bump libthrift to latest...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/132 --- 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 t

[1/2] incubator-distributedlog git commit: DL-204: Bump libthrift to latest version for distributedlog-core

2017-06-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 9c6c9c452 -> 0cb775c93 http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/0cb775c9/distributedlog-tutorials/distributedlog-basic/basic-2.md

[GitHub] incubator-distributedlog issue #134: Fix merge script issues

2017-06-12 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/134 /cc @fcuny @jiazhai --- 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] incubator-distributedlog pull request #134: Fix merge script issues

2017-06-12 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/134 Fix merge script issues Problem: Currently the merge script is broken when the reviewer's id is not in reviewers list. Solution: - Change "reviewers[review

[DRAFT] What the new Status Pages potentially look like

2017-06-12 Thread John D. Ament
All, We're piloting a new format for the podling status pages. Specifically, the current status page leaves a lot to be desired - it's basically crafted html, there's no structure and its hard to find missing items. The end goal is to have a web form editable in Whimsy, but until we get more inp

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

2017-06-12 Thread Edward Ribeiro
​+ ​1 ​ ​( ​n ​o ​n ​ ​b ​i ​n ​d ​i ​n ​g ​)​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ ​ On Fri, Jun 9, 2017 at 3:47 AM, Sijie Guo wrote: > (dropping general@) > > - Sijie > > On Thu, Jun 8, 2017 at 5:21 PM, Sijie Guo wrote: > > > ( /cc bookkeeper dev@ and incubator general@ for awareness ) > > > > Hi all