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:

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

[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

[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

[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

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

[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] [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:

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

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

[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

[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

[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

[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

[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=15745706#comment-15745706 ] 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

[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

[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

[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

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

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

[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

[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

[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

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

[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] [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 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] [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 #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-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 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] [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

[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

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

[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

[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

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

[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] [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 @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=15746663#comment-15746663 ] ASF GitHub Bot commented on DL-140: --- Github user sijie commented on the issue: https://

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

[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

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

[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

[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

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

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

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

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

[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

[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] [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://

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

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

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

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