incubator-distributedlog git commit: DL-198: Combining the NOTICE, LICENSE, and DISCLAIMER files

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 8fb0901e0 -> 2adb499f2 DL-198: Combining the NOTICE, LICENSE, and DISCLAIMER files Author: Sijie Guo <si...@apache.org> Reviewers: Henry Saputra <henry.sapu...@gmail.com>, Jia Zhai <zhaiji...@gmail

incubator-distributedlog git commit: DL-198: Combining the NOTICE, LICENSE, and DISCLAIMER files

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating d63e34459 -> 49c31cd53 DL-198: Combining the NOTICE, LICENSE, and DISCLAIMER files Author: Sijie Guo <si...@apache.org> Reviewers: Henry Saputra <henry.sapu...@gmail.com>, Ji

incubator-distributedlog git commit: Change the NOTICE year for Twitter Copyright to 2016

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 573b883fd -> 8fb0901e0 Change the NOTICE year for Twitter Copyright to 2016 Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com> Closes #127 from sijie/change_notice Project:

incubator-distributedlog git commit: Change the NOTICE year for Twitter Copyright to 2016

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating 1ddb5f4f6 -> d63e34459 Change the NOTICE year for Twitter Copyright to 2016 Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com> Closes #127 from sijie/change_

incubator-distributedlog git commit: DL-196: Remove 'Copyright 2017 The Apache Software Foundation' from license header

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating 1b70ac2d0 -> 1ddb5f4f6 DL-196: Remove 'Copyright 2017 The Apache Software Foundation' from license header Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com&g

incubator-distributedlog git commit: DL-196: Remove 'Copyright 2017 The Apache Software Foundation' from license header

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 58fd30e70 -> 573b883fd DL-196: Remove 'Copyright 2017 The Apache Software Foundation' from license header Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com> Closes #126 f

incubator-distributedlog git commit: DL-197: Add DISCLAIMER file to the project

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating 63ace6cbf -> 1b70ac2d0 DL-197: Add DISCLAIMER file to the project DISCLAIMER is missing for source release. Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com&g

incubator-distributedlog git commit: DL-197: Add DISCLAIMER file to the project

2017-04-12 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 1f705bea6 -> 58fd30e70 DL-197: Add DISCLAIMER file to the project DISCLAIMER is missing for source release. Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhaiji...@gmail.com> Closes #125 f

Re: [DISCUSS] Combining the NOTICE, LICENSE, and DISCLAIMER files

2017-04-11 Thread Sijie Guo
I created https://issues.apache.org/jira/browse/DL-198 for combining those asf files. https://github.com/apache/incubator-distributedlog/pull/128 is out for review. Can mentors take a look to make sure I am doing the right thing for asf files? - Sijie On Mon, Apr 10, 2017 at 11:52 PM, Sijie

[GitHub] incubator-distributedlog pull request #128: DL-198: Combining the NOTICE, LI...

2017-04-11 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/128 DL-198: Combining the NOTICE, LICENSE, and DISCLAIMER files You can merge this pull request into a Git repository by running: $ git pull https://github.com/sijie/incubator

Re: [DISCUSS] Combining the NOTICE, LICENSE, and DISCLAIMER files

2017-04-11 Thread Sijie Guo
I think this is doable - we can change the binary package assembly configuration (src/assemble/bin/xml) to include NOTICE/DISCLAIMER/LICENSE from root directly. Let me file one JIRA and send out a pull request for it. - Sijie On Mon, Apr 10, 2017 at 5:02 PM, Henry Saputra <henry.s

[GitHub] incubator-distributedlog pull request #127: Change the NOTICE year for Twitt...

2017-04-11 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/127 Change the NOTICE year for Twitter Copyright to 2016 You can merge this pull request into a Git repository by running: $ git pull https://github.com/sijie/incubator

Re: [VOTE] Apache DistributedLog release 0.4.0-incubating

2017-04-11 Thread Sijie Guo
Awesome. I learned a lot from this discussion. We will address these comments and call another vote. - Sijie On Mon, Apr 10, 2017 at 3:11 PM, Marvin Humphrey <mar...@rectangular.com> wrote: > On Mon, Apr 10, 2017 at 2:28 PM, Henry Saputra <henry.sapu...@gmail.com> > wrote

[GitHub] incubator-distributedlog issue #126: DL-196: Remove 'Copyright 2017 The Apac...

2017-04-10 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/126 /cc @fcuny @leighst @mgodave for review --- 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] incubator-distributedlog issue #125: DL-197: Add DISCLAIMER file to the proj...

2017-04-10 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/125 /cc @leighst @fcuny @mgodave for a review --- 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] incubator-distributedlog pull request #125: DL-197: Add DISCLAIMER file to t...

2017-04-10 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/125 DL-197: Add DISCLAIMER file to the project DISCLAIMER is missing for source release. You can merge this pull request into a Git repository by running: $ git pull https

[VOTE] Apache DistributedLog release 0.4.0-incubating

2017-04-05 Thread Sijie Guo
Hi everyone, Please review and vote on the release candidate #3 for the Apache DistributedLog version 0.4.0-incubating, as follows: [ ] +1, Approve the release [ ] -1, Do not approve the release (please provide specific comments) The complete staging area is available for your review,

Re: [VOTE] Release 0.4.0, release candidate #3

2017-04-05 Thread Sijie Guo
I am happy to announce that we have unanimously approved this release. There are 8 +1 votes and no disapproving votes. Thanks everyone! - Sijie On Tue, Mar 28, 2017 at 3:29 PM, Sijie Guo <guosi...@gmail.com> wrote: > Hi all, > > Please review and vote on the relea

Re: [VOTE] Release 0.4.0, release candidate #3

2017-04-04 Thread Sijie Guo
Can any mentors take a look at this release? If it looks good, I will try voting in incubator mail list. - Sijie On Tue, Apr 4, 2017 at 1:50 AM, Jay Juma <jayk.j...@gmail.com> wrote: > +1 (non-binding) > > > > On Tue, Mar 28, 2017 at 3:29 PM, Sijie Guo <guosi...@gmail.

incubator-distributedlog git commit: DL-195: exclude DISCLAIMER.bin.txt for Apache Rat Check Failures

2017-04-03 Thread sijie
0.050 s] [INFO] [INFO] BUILD SUCCESS ``` Author: jiazhai <zhaiji...@gmail.com> Author: jiazhai <jiaz...@users.noreply.github.com> Reviewers: Sijie Guo <si...@apache.org> Closes #124 from jiazhai/DL-195

[GitHub] incubator-distributedlog issue #27: DL-23: update libthrift version in distr...

2017-04-03 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/27 mind closing this PR? if it is inactive. --- 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] incubator-distributedlog issue #16: (WIP) Kafka publisher uses distributed l...

2017-04-03 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/16 mind closing this PR? if it is no active. --- 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] incubator-distributedlog issue #124: DL-195: exclude DISCLAIMER.bin.txt for ...

2017-04-03 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/124 LGTM +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 and wishes

Re: Apache BookKeeper Talk at #lfvault

2017-04-03 Thread Sijie Guo
Henry, (sorry I missed your email here) It went pretty well. There were quite a lot of attendees and quite a few questions at the end of talk. Hope it would help the community on BK and DL. - Sijie On Fri, Mar 24, 2017 at 4:28 PM, Henry Saputra <henry.sapu...@gmail.com> wrote: > Nice

Re: [VOTE] Release 0.4.0, release candidate #3

2017-03-29 Thread Sijie Guo
I think apache-rat:check will fail because of https://issues.apache.org/jira/browse/DL-195. But I don't think it is a blocker for the release. - Sijie On Tue, Mar 28, 2017 at 3:29 PM, Sijie Guo <guosi...@gmail.com> wrote: > Hi all, > > Please review and vote on the relea

incubator-distributedlog git commit: [maven-release-plugin] prepare for next development iteration

2017-03-28 Thread sijie
mit: 63ace6cbfed5c9b55cd761db6f0f430453ed3a7e Parents: 9c7368a Author: Sijie Guo <si...@apache.org> Authored: Tue Mar 28 14:18:54 2017 -0700 Committer: Sijie Guo <si...@apache.org> Committed: Tue Mar 28 14:18:54 2017 -0700 -- distributedlog-be

[incubator-distributedlog] Git Push Summary

2017-03-28 Thread sijie
Repository: incubator-distributedlog Updated Tags: refs/tags/v0.4.0-incubating-RC3_2.11 [created] 06e951490

incubator-distributedlog git commit: Release 0.4.0-incubating RC3 for scala 2.11

2017-03-28 Thread sijie
455 Parents: d5f2a81 Author: Sijie Guo <si...@apache.org> Authored: Tue Mar 28 14:02:29 2017 -0700 Committer: Sijie Guo <si...@apache.org> Committed: Tue Mar 28 14:02:29 2017 -0700 -- distributedlog-be

incubator-distributedlog git commit: [maven-release-plugin] prepare for next development iteration

2017-03-28 Thread sijie
mit: d5f2a817fec8dcc7b93211033bd8efe600b30c2f Parents: 2156e2a Author: Sijie Guo <si...@apache.org> Authored: Tue Mar 28 02:16:03 2017 -0700 Committer: Sijie Guo <si...@apache.org> Committed: Tue Mar 28 02:16:03 2017 -0700 -- distributedlog-be

[incubator-distributedlog] Git Push Summary

2017-03-28 Thread sijie
Repository: incubator-distributedlog Updated Tags: refs/tags/v0.4.0-incubating-RC3 [deleted] 1d6faf941

incubator-distributedlog git commit: [maven-release-plugin] prepare release v0.4.0-incubating-RC3

2017-03-28 Thread sijie
mit: 2156e2af71d5e34d488d933516f2d675ab1251cd Parents: 6e82ad2 Author: Sijie Guo <si...@apache.org> Authored: Tue Mar 28 02:14:04 2017 -0700 Committer: Sijie Guo <si...@apache.org> Committed: Tue Mar 28 02:14:04 2017 -0700 -- distributedlog-be

[incubator-distributedlog] Git Push Summary

2017-03-28 Thread sijie
Repository: incubator-distributedlog Updated Tags: refs/tags/v0.4.0-incubating-RC3 [created] 1d6faf941

incubator-distributedlog git commit: DL-184: reduce server_graceful_shutdown_period_ms from 35s to 20s, which is smaller than the default daemon stop timeout value

2017-03-28 Thread sijie
h Stewart <lstew...@apache.org>, Sijie Guo <si...@apache.org> Closes #115 from xieliang/DL-184-graceful Project: http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/commit/57e34779 Tree: http://

incubator-distributedlog git commit: DL-190: Add DISCLAIMER to the release packages

2017-03-22 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 27b4cddcc -> 2d2a4066c DL-190: Add DISCLAIMER to the release packages per apache release policy, we need to DISCLAIMER for releases. http://incubator.apache.org/incubation/Incubation_Policy.html#Releases Author: Si

incubator-distributedlog git commit: DL-191: Fix license header issues

2017-03-22 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating 264bd5ee2 -> 94e16762a DL-191: Fix license header issues the copyright year is wrong. Author: Sijie Guo <si...@apache.org> Reviewers: Leigh Stewart <lstew...@apache.org>, Flavi

incubator-distributedlog git commit: DL-191: Fix license header issues

2017-03-22 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master b4b0a7693 -> 27b4cddcc DL-191: Fix license header issues the copyright year is wrong. Author: Sijie Guo <si...@apache.org> Reviewers: Leigh Stewart <lstew...@apache.org>, Flavio Junqueira <f...@a

[GitHub] incubator-distributedlog issue #121: DL-191: Fix license header issues

2017-03-22 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/121 thank you @leighst . going to merge this one and resume the 0.4.0 release. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

incubator-distributedlog git commit: DL-193: Bumped up the Guava version

2017-03-22 Thread sijie
t;henry.sapu...@gmail.com>, Sijie Guo <si...@apache.org> Closes #123 from arvindkandhare/guava_version (cherry picked from commit b4b0a7693c5095372b8bc00baf5f0139708a1902) Signed-off-by: Sijie Guo <si...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/r

incubator-distributedlog git commit: DL-193: Bumped up the Guava version

2017-03-21 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 132b68253 -> b4b0a7693 DL-193: Bumped up the Guava version Moving the Guava version from 16.0 to 20.0. Author: arvindkandhare <arvind.kandh...@emc.com> Reviewers: Henry Saputra <henry.sapu...@gmail.com>

[GitHub] incubator-distributedlog issue #123: DL-193: Bumped up the Guava version

2017-03-18 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/123 LGTM +1. Thanks @arvindkandhare --- 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] incubator-distributedlog issue #123: DL-193: Bumped up the Guava version

2017-03-17 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/123 @arvindkandhare - I merged your change in bookkeeper and released a version - 4.3.7-TWTTR-OSS Can you update your pull request for this? --- If your project is set up

Re: release 0.4.0

2017-03-17 Thread Sijie Guo
Got back on the release. Will cut a new release RC on the weekend. - Sijie On Mon, Feb 13, 2017 at 10:00 AM, Sijie Guo <si...@apache.org> wrote: > It seems that we already got all the issues addressed in the pull > requests. I will try to cut a new release candidate soon. (I

[GitHub] incubator-distributedlog issue #123: DL-193: Bumped up the Guava version

2017-03-15 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/123 gotcha. Is it possible to shade the guava in bookkeeper? --- 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

[GitHub] incubator-distributedlog issue #123: DL-193: Bumped up the Guava version

2017-03-14 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/123 @arvindkandhare can you just exclude "guava" from the bookkeeper dependency and use 0.20 here? I am trying to understand if we really need a new bk version from twit

[GitHub] incubator-distributedlog issue #123: DL-193: Bumped up the Guava version

2017-03-08 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/123 Seems both travis CI and Jenkins failed with similar error ` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test

Re: release 0.4.0

2017-02-13 Thread Sijie Guo
It seems that we already got all the issues addressed in the pull requests. I will try to cut a new release candidate soon. (I am traveling these days, so my time is not very guaranteed. I will try to do my best to cut an RC). - Sijie On Mon, Jan 30, 2017 at 6:22 PM, Sijie Guo <si...@apache.

[GitHub] incubator-distributedlog pull request #121: DL-191: Fix license header issue...

2017-02-07 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/121#discussion_r13964 --- Diff: distributedlog-core/conf/zookeeper.conf.dynamic.template --- @@ -1 +1 @@ -#/**# * Copyright 2007 The Apache Software

incubator-distributedlog git commit: DL-192: year needs updating in NOTICE

2017-02-07 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 41c7825b4 -> 132b68253 DL-192: year needs updating in NOTICE The year was 2016. but we are going to release at 2017. so updating the files to 2017. Author: Sijie Guo <si...@apache.org> Reviewers: Leigh Stewa

incubator-distributedlog git commit: DL-192: year needs updating in NOTICE

2017-02-07 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating da80b093b -> 4c327458f DL-192: year needs updating in NOTICE The year was 2016. but we are going to release at 2017. so updating the files to 2017. Author: Sijie Guo <si...@apache.org>

[GitHub] incubator-distributedlog issue #119: DL-192: year needs updating in NOTICE

2017-02-07 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/119 @fpj yeah. will first add a bullet to release guide for 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

[GitHub] incubator-distributedlog pull request #120: DL-190: Add DISCLAIMER to the re...

2017-02-07 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/120#discussion_r13567 --- Diff: src/main/resources/DISCLAIMER.bin.txt --- @@ -0,0 +1 @@ +Apache DistributedLog is an effort undergoing incubation

incubator-distributedlog git commit: Fix log4j binding issue on binary package

2017-02-07 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 1b36687b6 -> 41c7825b4 Fix log4j binding issue on binary package Author: Sijie Guo <si...@apache.org> Reviewers: Leigh Stewart <lstew...@apache.org> Closes #122 from sijie/sijie/fix_log4j_issue Proj

[GitHub] incubator-distributedlog pull request #121: DL-191: Fix license header issue...

2017-02-02 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/121#discussion_r99267093 --- Diff: distributedlog-core/conf/zookeeper.conf.dynamic.template --- @@ -1 +1 @@ -#/**# * Copyright 2007 The Apache Software Foundation

Re: DistributedLog Podling Report Draft - Feb 2017

2017-02-01 Thread Sijie Guo
I updated this to the incubator wiki page - https://wiki.apache.org/incubator/February2017 Please take a look and let me know if you have any questions. Can mentors take a look and sign off? - Sijie On Wed, Feb 1, 2017 at 1:10 PM, Sijie Guo <si...@apache.org> wrote: > If there is no

[GitHub] incubator-distributedlog issue #119: DL-192: year needs updating in NOTICE

2017-01-31 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/119 I think this has to be updated on every new release to use current year. I will check if there is any plugin can be used for automating this. --- If your project is set up

[GitHub] incubator-distributedlog issue #122: Fix log4j binding issue on binary packa...

2017-01-30 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/122 ^ @xieliang --- 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

release 0.4.0

2017-01-30 Thread Sijie Guo
-distributedlog/pull/121 We need to address these issues and cut another RC. Can anyone review these? It would be great if the mentors can take a look at those fixes too. - Sijie

[GitHub] incubator-distributedlog pull request #121: DL-191: Fix license header issue...

2017-01-30 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/121 DL-191: Fix license header issues the copyright year is wrong. You can merge this pull request into a Git repository by running: $ git pull https://github.com/sijie/incubator

[GitHub] incubator-distributedlog pull request #120: DL-191: Add DISCLAIMER to the re...

2017-01-30 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/120 DL-191: Add DISCLAIMER to the release packages per apache release policy, we need to DISCLAIMER for releases. http://incubator.apache.org/incubation/Incubation_Policy.html

[GitHub] incubator-distributedlog pull request #119: DL-192: year needs updating in N...

2017-01-30 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/119 DL-192: year needs updating in NOTICE The year was 2016. but we are going to release at 2017. so updating the files to 2017. You can merge this pull request into a Git repository

incubator-distributedlog git commit: DL-186: fix common.sh grep bug; kill MultiReader java process as expected

2017-01-30 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating c2654129f -> b35e29c47 DL-186: fix common.sh grep bug; kill MultiReader java process as expected Author: xieliang <xieliang...@gmail.com> Reviewers: Sijie Guo <si...@apache.org> C

incubator-distributedlog git commit: DL-186: fix common.sh grep bug; kill MultiReader java process as expected

2017-01-30 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master d5ce69e85 -> 1b36687b6 DL-186: fix common.sh grep bug; kill MultiReader java process as expected Author: xieliang <xieliang...@gmail.com> Reviewers: Sijie Guo <si...@apache.org> Closes #114 from

incubator-distributedlog git commit: DL-187: load test doc typo fix

2017-01-30 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/release-0.4.0-incubating f0050dbc1 -> c2654129f DL-187: load test doc typo fix Author: xieliang <xieliang...@gmail.com> Reviewers: Sijie Guo <si...@apache.org> Closes #117 from xieliang/DL-187-doc_typo (che

incubator-distributedlog git commit: DL-189: remove the extra NOTICE change

2017-01-30 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 732141c72 -> 6f7feb01f DL-189: remove the extra NOTICE change Author: xieliang <xieliang...@gmail.com> Reviewers: Sijie Guo <si...@apache.org> Closes #118 from xieliang/DL-189 Project: http://git-wi

Re: [VOTE] Apache DistributedLog release 0.4.0-incubating

2017-01-26 Thread Sijie Guo
Gotcha. Cancelling the vote. Will address the concerns in new RC. Thanks John! - Sijie On Thu, Jan 26, 2017 at 5:46 PM, John D. Ament <johndam...@apache.org> wrote: > On Thu, Jan 26, 2017 at 8:27 PM Sijie Guo <si...@apache.org> wrote: > > > On Thu, Jan 26, 2017 a

Re: [VOTE] Apache DistributedLog release 0.4.0-incubating

2017-01-26 Thread Sijie Guo
e doing when we are making a similar code change. And there is a JIRA to follow up after this release. Do you see a major concern here? If so, we will address it in the new RC. Thanks, Sijie > > On Thu, Jan 26, 2017 at 1:36 AM Sijie Guo <si...@apache.org> wrote: > > > Hi

[VOTE] Apache DistributedLog release 0.4.0-incubating

2017-01-25 Thread Sijie Guo
Hi everyone, Please review and vote on the release candidate #2 for the Apache DistributedLog version 0.4.0-incubating, as follows: [ ] +1, Approve the release [ ] -1, Do not approve the release (please provide specific comments) The complete staging area is available for your

Re: [RESULT] [VOTE] Release 0.4.0, release candidate #2

2017-01-25 Thread Sijie Guo
Not yet. Unfortunately I think I need to run another vote in gene...@incubator.apache.org since we are still in incubation. I will start a vote thread in general@ today. - Sijie On Wed, Jan 25, 2017 at 12:21 AM, Asko Kauppi <asko.kau...@zalando.fi> wrote: > Is there a URL / newsfe

[RESULT] [VOTE] Release 0.4.0, release candidate #2

2017-01-24 Thread Sijie Guo
I am happy to announce that we have unanimously approved this release. There are 8 +1 votes and no disapproving votes. Thanks everyone! - Sijie On Tue, Jan 10, 2017 at 11:56 PM, Sijie Guo <si...@apache.org> wrote: > Hi all, > > Please review and vote on the relea

[GitHub] incubator-distributedlog issue #117: DL-187 : load test doc typo fix

2017-01-24 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/117 LGTM +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 and wishes

Re: [VOTE] Release 0.4.0, release candidate #2

2017-01-18 Thread Sijie Guo
t; >> +1 >> >> LGTM. compiled the source packages and ran dbench. the license files look >> good. >> >> - jd >> >> On Tue, Jan 10, 2017 at 11:56 PM, Sijie Guo <si...@apache.org> wrote: >> >>> Hi all, >>> >>> Plea

Re: [VOTE] Release 0.4.0, release candidate #2

2017-01-12 Thread Sijie Guo
On Jan 12, 2017 6:59 PM, "liang xie" <xieliang...@gmail.com> wrote: I am not good at here, per my knowledge, we should add the bindings into benchmark package, at least in next release. Agree. Do you mind creating a Jira for this? On Fri, Jan 13, 2017 at 12:42 AM, Sijie Guo &

Re: [VOTE] Release 0.4.0, release candidate #2

2017-01-12 Thread Sijie Guo
I think we turned the log4j binding scope to test sometime ago. The thought was not to conflict bindings when DL is used as a library in other projects. For binaries, it seems worth brining the bindings in. What does other projects do for slf4j bindings? Sijie On Jan 12, 2017 12:58 AM, "

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-12 Thread Sijie Guo
Yeah, currently some of the tests are time dependent. So they failed sometimes when the machine is slow. There were efforts on improving this and will also do this in next release. Sijie On Jan 12, 2017 3:32 AM, "Flavio Junqueira" <f...@apache.org> wrote: > Ok, so I've seen

[VOTE] Release 0.4.0, release candidate #2

2017-01-10 Thread Sijie Guo
be open for at least 72 hours. It is adopted by majority approval, with at least 3 PPMC affirmative votes. Thanks, Sijie [1] https://issues.apache.org/jira/secure/ReleaseNote.jspa? projectId=12320620=12337980 [2] https://dist.apache.org/repos/dist/dev/incubator/distributedlog/0

incubator-distributedlog git commit: DL-183: Configure DL to generate source jar

2017-01-10 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master c5e921efe -> 732141c72 DL-183: Configure DL to generate source jar also move the plugin version definition to the properties Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhai...@apache.org&g

incubator-distributedlog git commit: [maven-release] prepare release 0.4.0-incubating 2.11

2017-01-10 Thread sijie
mit: 8092373d0d4994f6d9e26e86951b53cea0c3937b Parents: 84e7eef Author: Sijie Guo <si...@apache.org> Authored: Tue Jan 10 21:52:23 2017 -0800 Committer: Sijie Guo <si...@apache.org> Committed: Tue Jan 10 21:52:23 2017 -0800 -- distributedlog-be

incubator-distributedlog git commit: Fix the script to make sure binary package can execute scripts correctly

2017-01-10 Thread sijie
ute - change the type of runner and copy the runner script to each tutorial module Author: Sijie Guo <si...@apache.org> Reviewers: Jia Zhai <zhai...@apache.org> Closes #111 from sijie/sijie/copy_all_scripts_to_modules Project: http://git-wip-us.apache.org/repos/asf/incubator-dist

incubator-distributedlog git commit: DL-182: assembled package should include README.md

2017-01-10 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 0c6e79497 -> bd13f74c5 DL-182: assembled package should include README.md Author: xieliang <xieliang...@gmail.com> Reviewers: Sijie Guo <si...@apache.org> Closes #112 from xieliang/DL-182-assembled

[GitHub] incubator-distributedlog issue #112: DL-182 : assembled package should inclu...

2017-01-10 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/112 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

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-10 Thread Sijie Guo
Liang had a pull request for the assembly issue. Going to cancel this vote and produce a new RC. Flavio, what are the tests failed in your case? Sijie On Jan 10, 2017 9:10 AM, "Flavio Junqueira" <f...@apache.org> wrote: Yeah, I can't build because of the same issue:

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-09 Thread Sijie Guo
On Mon, Jan 9, 2017 at 5:32 PM, Sijie Guo <si...@apache.org> wrote: > Thank you for your suggestion, Khurrum. Comment inline. I'd like to > collect more feedbacks and then cut another RC. > > > On Mon, Jan 9, 2017 at 11:43 AM, Khurrum Nasim <khurrumnas...@gmail.com>

[GitHub] incubator-distributedlog pull request #111: Fix the script to make sure bina...

2017-01-09 Thread sijie
GitHub user sijie opened a pull request: https://github.com/apache/incubator-distributedlog/pull/111 Fix the script to make sure binary package can execute scripts correctly - fix the release jar path - copy the common script to each modules otherwise the built package won't

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-09 Thread Sijie Guo
ranch. I merged your change to both master and release-0.4.0-incubating branch. so it should be okay now. > On Mon, Jan 9, 2017 at 12:04 PM, Sijie Guo <si...@apache.org> wrote: > > Hi all, > > > > Please review and vote on the release candidate #1 for the version 0.4.0, > &

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-09 Thread Sijie Guo
On Mon, Jan 9, 2017 at 11:07 PM, Sijie Guo <si...@apache.org> wrote: > > > On Mon, Jan 9, 2017 at 8:00 PM, liang xie <xieliang...@gmail.com> wrote: > >> Glad to see 0.4 will be release soon :) >> I could not find tutorials package from link[2], it should be re

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-09 Thread Sijie Guo
Failed to create assembly: Error > >> adding file to archive: > >> /tmp/distributedlog-0.4.0-incubating/distributedlog- > service/../README.md > >> isn't a file. -> [Help 1] > >> > >> And none of you saw this : DL-181, it can be reproduc

[GitHub] incubator-distributedlog issue #110: DL-181 : avoid starting failure due to ...

2017-01-09 Thread sijie
Github user sijie commented on the issue: https://github.com/apache/incubator-distributedlog/pull/110 merged to master and also cherry-pick it to release-0.4.0-incubating branch --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

incubator-distributedlog git commit: DL-181: avoid starting failure due to call Optional.get() on an absent value

2017-01-09 Thread sijie
Reviewers: Sijie Guo <si...@apache.org> Closes #110 from xieliang/DL-181-startingProxyFailed (cherry picked from commit 0c6e7949718abf5c30c150ffd045a8b765373cd6) Signed-off-by: Sijie Guo <si...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/repo Co

incubator-distributedlog git commit: DL-181: avoid starting failure due to call Optional.get() on an absent value

2017-01-09 Thread sijie
Repository: incubator-distributedlog Updated Branches: refs/heads/master 8bb9cc004 -> 0c6e79497 DL-181: avoid starting failure due to call Optional.get() on an absent value see original JIRA for the detailed error msg Author: xieliang <xieliang...@gmail.com> Reviewers: Siji

Re: [VOTE] Release 0.4.0, release candidate #1

2017-01-09 Thread Sijie Guo
Thank you for your suggestion, Khurrum. Comment inline. I'd like to collect more feedbacks and then cut another RC. On Mon, Jan 9, 2017 at 11:43 AM, Khurrum Nasim <khurrumnas...@gmail.com> wrote: > Thanks Sijie for working on this. A couple of comments: > > - do we need to pu

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101716 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101711 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101702 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101707 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101705 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

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

2017-01-08 Thread sijie
Github user sijie commented on a diff in the pull request: https://github.com/apache/incubator-distributedlog/pull/51#discussion_r95101701 --- Diff: website/community/release_guide.md --- @@ -0,0 +1,507 @@ +--- +layout: default +title: "DistributedLog Release

[VOTE] Release 0.4.0, release candidate #1

2017-01-08 Thread Sijie Guo
s adopted by majority approval, with at least 3 PPMC affirmative votes. Thanks, Sijie [1] https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12320620=12337980 [2] https://dist.apache.org/repos/dist/dev/incubator/distributedlog/0.4.0-incubating/ [3] https://repository.a

[11/51] [partial] incubator-distributedlog git commit: [release 0.4.0-incubating] Add Java API docs

2017-01-08 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/1e6843e5/content/docs/0.4.0-incubating/api/java/org/apache/distributedlog/client/routing/TwitterServerSetWatcher.html -- diff --git

[29/51] [partial] incubator-distributedlog git commit: [release 0.4.0-incubating] Add Java API docs

2017-01-08 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/1e6843e5/content/docs/0.4.0-incubating/api/java/org/apache/distributedlog/class-use/DistributedLogConfiguration.html -- diff --git

[27/51] [partial] incubator-distributedlog git commit: [release 0.4.0-incubating] Add Java API docs

2017-01-08 Thread sijie
http://git-wip-us.apache.org/repos/asf/incubator-distributedlog/blob/1e6843e5/content/docs/0.4.0-incubating/api/java/org/apache/distributedlog/class-use/LogRecord.Reader.html -- diff --git

<    1   2   3   4   5   6   7   8   >