Re: Slack channel for DistributedLog

2016-12-13 Thread Sijie Guo
+ dev Gerrit, I asked in the private list. there is official support for Slack or Gitter for the ASF projects. So we are able to setup an official Slack or Gitter channel. Let me call a discussion thread to see what's people's opinions. - Sijie On Dec 11, 2016 5:44 PM, "Sijie Guo" wrote: G

Introduction - wanting to work on the documentation

2016-12-13 Thread Asko Kauppi
Hi all, my name is Asko Kauppi and I work as a technical Agility Coach at the Zalando Helsinki site we started in 2015. I'm interested in persistent message buses in general, because I see them offering a new kind of way to do microservices. Instead of each service being responsible for its state

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15747138#comment-15747138 ] ASF GitHub Bot commented on DL-140: --- Github user xieliang commented on the issue: https

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread xieliang
Github user xieliang commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 It's not the same thing. The travis ci runs with "findbugs-maven-plugin", and i ran with findbugs binary package. (From http://gleclaire.github.io/findbugs-maven-plugin/index.ht

[jira] [Created] (DL-142) Need a CI job for integration tests

2016-12-13 Thread Sijie Guo (JIRA)
Sijie Guo created DL-142: Summary: Need a CI job for integration tests Key: DL-142 URL: https://issues.apache.org/jira/browse/DL-142 Project: DistributedLog Issue Type: Task Components: bui

[jira] [Created] (DL-141) DLog scripts are broken

2016-12-13 Thread Sijie Guo (JIRA)
Sijie Guo created DL-141: Summary: DLog scripts are broken Key: DL-141 URL: https://issues.apache.org/jira/browse/DL-141 Project: DistributedLog Issue Type: Bug Components: distributedlog-c

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746993#comment-15746993 ] ASF GitHub Bot commented on DL-140: --- Github user sijie commented on the issue: https://

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 @xieliang I think currently we are using 3.0.3 in the pom.xml file. ` org.codehaus.mojo findbugs-maven-plugin 3.0.3

[jira] [Updated] (DL-123) Every tests should have timeout

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-123: - Labels: help-wanted (was: ) > Every tests should have timeout > --- > > K

[jira] [Commented] (DL-123) Every tests should have timeout

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746983#comment-15746983 ] Sijie Guo commented on DL-123: -- [~boyork] I just assigned this jira to you. Look forward to your

[jira] [Updated] (DL-123) Every tests should have timeout

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-123: - Assignee: Brennon York > Every tests should have timeout > --- > > Key: DL

Re: Intro

2016-12-13 Thread Sijie Guo
Welcome Brennon! We followed most of the other apache projects are doing. If you have been working with other apache projects, you will be totally fine with DL community. In general, you can check the developer guide https://cwiki.apache.org/confluence/display/DL/Developer+Guide We use Jira for t

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746952#comment-15746952 ] ASF GitHub Bot commented on DL-140: --- Github user xieliang commented on the issue: https

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread xieliang
Github user xieliang commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 I ran the findbug (3.0.1) binary on my local devbox. Those multi-thread warings exist even after importing the findbug exclude xml file manually, what's the findbugs binary version

[jira] [Updated] (DL-136) Avoid empty catch blocks

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-136?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-136: - Assignee: Brennon York > Avoid empty catch blocks > > > Key: DL-136 >

[jira] [Commented] (DL-136) Avoid empty catch blocks

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746933#comment-15746933 ] Sijie Guo commented on DL-136: -- Cool. I just assigned this to you. I also added you to the contri

Intro

2016-12-13 Thread York, Brennon
Hi all, I wanted to introduce myself and say that after seeing the incubator for this project it sounds exciting! Definitely something I’d like to get more exposure to as it would be very applicable to the team I lead. That said I’d like to start contributing and wondering how you all do that. I

[jira] [Commented] (DL-123) Every tests should have timeout

2016-12-13 Thread Brennon York (JIRA)
[ https://issues.apache.org/jira/browse/DL-123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746861#comment-15746861 ] Brennon York commented on DL-123: - Hi, I'd like to begin contributing to this project and woul

[jira] [Commented] (DL-136) Avoid empty catch blocks

2016-12-13 Thread Brennon York (JIRA)
[ https://issues.apache.org/jira/browse/DL-136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746856#comment-15746856 ] Brennon York commented on DL-136: - Hi, I'd like to start contributing to this project and woul

[GitHub] incubator-distributedlog issue #55: Hack week cleanup

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/55 @franckcuny the travis ci passed. looking into apache ci now. --- 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 pr

[GitHub] incubator-distributedlog issue #55: Hack week cleanup

2016-12-13 Thread franckcuny
Github user franckcuny commented on the issue: https://github.com/apache/incubator-distributedlog/pull/55 This is actually still broken. --- 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 f

[GitHub] incubator-distributedlog pull request #72: Pr docs introduction

2016-12-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/72 --- 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 th

[GitHub] incubator-distributedlog issue #72: Pr docs introduction

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/72 looks good to me. +1 --- 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

incubator-distributedlog git commit: Pr docs introduction

2016-12-13 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 5eb5434f6 -> fd63b34bc Pr docs introduction Small suggestions to basics/introduction doc Author: Asko Kauppi Reviewers: Sijie Guo Closes #72 from akauppi/pr-docs-introduction Project: http://git-wip-us.apache.org/r

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746663#comment-15746663 ] ASF GitHub Bot commented on DL-140: --- Github user sijie commented on the issue: https://

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 @xieliang the change looks good. but I am interested to see how did you find this. I thought we already enable findbugs on ci jobs. I am wondering why didn't we detect the findbug warn

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746660#comment-15746660 ] ASF GitHub Bot commented on DL-140: --- Github user sijie commented on the issue: https://

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 @franckcuny I passed the travis ci for both scala 2.10 and 2.11. it should be transient errors. --- If your project is set up for it, you can reply to this email and have your reply a

[jira] [Updated] (DL-49) Support building project with scala 2.10 and 2.11

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-49: Fix Version/s: (was: 0.5.0) 0.4.0 > Support building project with scala 2.10 and 2.11 > --

[jira] [Updated] (DL-85) Publish both per-stream stats and aggregation stats

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-85?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-85: Fix Version/s: (was: 0.5.0) 0.4.0 > Publish both per-stream stats and aggregation stats >

[jira] [Resolved] (DL-135) Cleanup System.out and System.err usage

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-135?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved DL-135. -- Resolution: Fixed Fix Version/s: 0.4.0 Merged the change. Thanks [~xieliang007] for the contribution. > Clea

[jira] [Updated] (DL-134) Add DistributedLog User MailList to pom file

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-134?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated DL-134: - Fix Version/s: (was: 0.5.0) 0.4.0 > Add DistributedLog User MailList to pom file >

[jira] [Commented] (DL-135) Cleanup System.out and System.err usage

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746651#comment-15746651 ] ASF GitHub Bot commented on DL-135: --- Github user asfgit closed the pull request at: htt

[GitHub] incubator-distributedlog pull request #70: DL-135 : cleanup System.out

2016-12-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/70 --- 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 th

incubator-distributedlog git commit: DL-135: cleanup System.out

2016-12-13 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 3a4c1608f -> 5eb5434f6 DL-135: cleanup System.out Author: xieliang Reviewers: Xi Liu , Franck Cuny Closes #70 from xieliang/DL-135 Project: http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/repo Commit

[jira] [Commented] (DL-134) Add DistributedLog User MailList to pom file

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-134?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746645#comment-15746645 ] ASF GitHub Bot commented on DL-134: --- Github user asfgit closed the pull request at: htt

[jira] [Resolved] (DL-134) Add DistributedLog User MailList to pom file

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-134?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved DL-134. -- Resolution: Fixed Fix Version/s: (was: 0.4.0) 0.5.0 Issue resolved by merging pull req

[GitHub] incubator-distributedlog pull request #56: DL-134: Add distributedlog user m...

2016-12-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/56 --- 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 th

incubator-distributedlog git commit: DL-134: Add distributedlog user mail list to pom file

2016-12-13 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master a645c311b -> 3a4c1608f DL-134: Add distributedlog user mail list to pom file Author: Sijie Guo Reviewers: Franck Cuny Closes #56 from sijie/sijie/DL-134 Project: http://git-wip-us.apache.org/repos/asf/incubator-dist

[jira] [Commented] (DL-95) Assign host ip client id if the client id is undefined

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746640#comment-15746640 ] ASF GitHub Bot commented on DL-95: -- Github user sijie commented on the issue: https://git

[GitHub] incubator-distributedlog issue #68: DL-95: Assign host ip client id if the c...

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/68 @franckcuny yup the change is accumulative changes. I didn't find a solution to send pull request based on other changes. The changes from DL-85 to DL-97 are accumulative. That

[jira] [Commented] (DL-86) Improve handling of lock conflicts in zk session lock

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-86?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746633#comment-15746633 ] ASF GitHub Bot commented on DL-86: -- Github user sijie commented on the issue: https://git

[GitHub] incubator-distributedlog issue #58: DL-86: Improve handling of lock conflict...

2016-12-13 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/58 @franckcuny I think it is a transient error. The tests passed on travis-ci but failed on apache jenkins ci. btw, for now, every pull request will be validated on both travis c

[jira] [Resolved] (DL-85) Publish both per-stream stats and aggregation stats

2016-12-13 Thread Sijie Guo (JIRA)
[ https://issues.apache.org/jira/browse/DL-85?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved DL-85. - Resolution: Fixed Fix Version/s: (was: 0.4.0) 0.5.0 Issue resolved by merging pull reques

[GitHub] incubator-distributedlog pull request #57: DL-85: Publish both per-stream st...

2016-12-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-distributedlog/pull/57 --- 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 th

[jira] [Commented] (DL-85) Publish both per-stream stats and aggregation stats

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-85?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746627#comment-15746627 ] ASF GitHub Bot commented on DL-85: -- Github user asfgit closed the pull request at: https:

incubator-distributedlog git commit: DL-85: Publish both per-stream stats and aggregation stats

2016-12-13 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 3fceccc6c -> a645c311b DL-85: Publish both per-stream stats and aggregation stats merge twittter's changes from Jordan Bull. Author: Jordan Bull Reviewers: Sijie Guo , Franck Cuny Closes #57 from sijie/merge/DL-85

[jira] [Commented] (DL-28) generate java code failed using scrooge

2016-12-13 Thread Tracy Martin (JIRA)
[ https://issues.apache.org/jira/browse/DL-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746252#comment-15746252 ] Tracy Martin commented on DL-28: /* */ also works. > generate java code failed using scrooge >

[jira] [Commented] (DL-28) generate java code failed using scrooge

2016-12-13 Thread Tracy Martin (JIRA)
[ https://issues.apache.org/jira/browse/DL-28?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746235#comment-15746235 ] Tracy Martin commented on DL-28: I can confirm that I see this issue on Windows. It appears to

[GitHub] incubator-distributedlog pull request #72: Pr docs introduction

2016-12-13 Thread akauppi
GitHub user akauppi opened a pull request: https://github.com/apache/incubator-distributedlog/pull/72 Pr docs introduction Small suggestions to basics/introduction doc You can merge this pull request into a Git repository by running: $ git pull https://github.com/akauppi/incuba

[GitHub] incubator-distributedlog issue #68: DL-95: Assign host ip client id if the c...

2016-12-13 Thread franckcuny
Github user franckcuny commented on the issue: https://github.com/apache/incubator-distributedlog/pull/68 @sijie this is a large change and the tests are failing. Is there anyway to reduce the scope of it ? It's hard to review. --- If your project is set up for it, you can reply to t

[jira] [Commented] (DL-95) Assign host ip client id if the client id is undefined

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745731#comment-15745731 ] ASF GitHub Bot commented on DL-95: -- Github user franckcuny commented on the issue: https:

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745729#comment-15745729 ] ASF GitHub Bot commented on DL-140: --- Github user franckcuny commented on the issue: htt

[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread franckcuny
Github user franckcuny commented on the issue: https://github.com/apache/incubator-distributedlog/pull/71 @xieliang could you verify if the failure is due to your change ? Is the test passing on your local environment ? cc @sijie --- If your project is set up for it, you ca

[jira] [Commented] (DL-86) Improve handling of lock conflicts in zk session lock

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-86?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745713#comment-15745713 ] ASF GitHub Bot commented on DL-86: -- Github user franckcuny commented on the issue: https:

[GitHub] incubator-distributedlog issue #58: DL-86: Improve handling of lock conflict...

2016-12-13 Thread franckcuny
Github user franckcuny commented on the issue: https://github.com/apache/incubator-distributedlog/pull/58 @sijie is this a transient error ? Looks like time out while doing some operation with ZK, so could be a misconfigured test too. --- If your project is set up for it, you can rep

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r9009 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745705#comment-15745705 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r92220975 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745707#comment-15745707 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745706#comment-15745706 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745704#comment-15745704 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r92221776 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745701#comment-15745701 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[jira] [Commented] (DL-81) Build the distributedlog release procedure

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745703#comment-15745703 ] ASF GitHub Bot commented on DL-81: -- Github user franckcuny commented on a diff in the pull req

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r92221264 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r92221508 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-13 Thread franckcuny
Github user franckcuny commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r92221057 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release Guide

[DISCUSS] DP-6 - CAS Style Writes

2016-12-13 Thread Jon Derrick
Hi, I've been thinking about a DP to provide a CAS style write capability in DL. If you are interested, please check out: https://cwiki.apache.org/confluence/display/DL/DP-6+-+CAS+Style+Writes -- - jderrick

[jira] [Commented] (DL-97) Remove unused methods in BKLogHandler

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-97?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745659#comment-15745659 ] ASF GitHub Bot commented on DL-97: -- Github user franckcuny commented on the issue: https:

[GitHub] incubator-distributedlog issue #69: DL-97: Remove unused methods in BKLogHan...

2016-12-13 Thread franckcuny
Github user franckcuny commented on the issue: https://github.com/apache/incubator-distributedlog/pull/69 @sijie the build is failing with the following error: ``` Results : Failed tests: testPositionUpdatesOnlyAfterWriteCompletionWithoutFsync(com.twitter.distri

[jira] [Commented] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread Liang Xie (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15744787#comment-15744787 ] Liang Xie commented on DL-140: -- passed "mvn test" under distributedlog-core dir on my box: "Tests

[jira] [Updated] (DL-140) Fix several FindBug "Inconsistent synchronization" warnings

2016-12-13 Thread Liang Xie (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Liang Xie updated DL-140: - Summary: Fix several FindBug "Inconsistent synchronization" warnings (was: Fix several "Inconsistent synchronization"

[jira] [Commented] (DL-140) Fix several "Inconsistent synchronization"s from FindBug warning

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15744766#comment-15744766 ] ASF GitHub Bot commented on DL-140: --- GitHub user xieliang opened a pull request: https:

[GitHub] incubator-distributedlog pull request #71: DL-140 : Fix distributedlog-core ...

2016-12-13 Thread xieliang
GitHub user xieliang opened a pull request: https://github.com/apache/incubator-distributedlog/pull/71 DL-140 : Fix distributedlog-core findbug inconsistent synchronization warings You can merge this pull request into a Git repository by running: $ git pull https://github.com

[jira] [Created] (DL-140) Fix several "Inconsistent synchronization"s from FindBug warning

2016-12-13 Thread Liang Xie (JIRA)
Liang Xie created DL-140: Summary: Fix several "Inconsistent synchronization"s from FindBug warning Key: DL-140 URL: https://issues.apache.org/jira/browse/DL-140 Project: DistributedLog Issue Type:

[jira] [Commented] (DL-135) Cleanup System.out and System.err usage

2016-12-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/DL-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15744645#comment-15744645 ] ASF GitHub Bot commented on DL-135: --- Github user xiliuant commented on the issue: https

[GitHub] incubator-distributedlog issue #70: DL-135 : cleanup System.out

2016-12-13 Thread xiliuant
Github user xiliuant commented on the issue: https://github.com/apache/incubator-distributedlog/pull/70 the change looks good to me +1 --- 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 fea

[VOTE] DP-2: Epoch Write Support

2016-12-13 Thread Xi Liu
I'd like to start voting on DP-2: https://cwiki.apache.org/confluence/display/DL/DP-2+-+Epoch+Write+Support The discussion email thread can be found: link Thank you for your feedback. - Xi

Re: "Code Analysis of Twitter’s DistributedLog"

2016-12-13 Thread Xi Liu
It is a really interesting analysis. It would definitely help DL improve the code quality. - Xi On Mon, Dec 12, 2016 at 5:43 PM, Sijie Guo wrote: > There is a blog post about "code analysis of distributedlog > ". It has > a few go

Re: vote process for proposals?

2016-12-13 Thread Xi Liu
Thank you Sijie. I feel it is good to have a voting process, so that it would be good to track if a proposal is accepted for developing or discarded due to any reasons. I will start with my proposal and see how it is going with the community. - Xi On Thu, Dec 8, 2016 at 9:11 PM, Sijie Guo wrote: