You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by Jun Liu <li...@apache.org> on 2018/05/18 09:26:45 UTC

[VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Hello All,

This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.

The Apache Dubbo community has voted on and approved a proposal to release Apache Dubbo (Incubating) version 2.6.2.

We now kindly request the Incubator PMC members review and vote on this incubator release.

Apache Dubbo™ (incubating) is a high-performance, java based, open source RPC framework. Dubbo offers three key functionalities, which include interface based remote call, fault tolerance & load balancing, and automatic service registration & discovery. 

Dubbo vote thread:
https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd971bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E

Dubbo vote result thread:
https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E

The release candidates:
https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2

Git tag for the release:
https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2 

Hash for the release tag:
afab04c53edab38d52275d2a198ea1aff7a4f41e

Release Notes:
https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md

The artifacts have been signed with Key : 28681CB1, which can be found in the keys file:
https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS

The vote will be open for at least 72 hours or until necessary number of votes are reached.

Please vote accordingly:
[ ] +1 approve 
[ ] +0 no opinion 
[ ] -1 disapprove with the reason

Thanks.
Jun Liu,
The Apache Dubbo (Incubating) Team


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Jun Liu <li...@apache.org>.
Hi All,

I am canceling this vote and prepare for a new release candidate RC2. Thanks for Willem, Ian and Justin for pointing out the issues in this candidate, we expect to address the following issues RC2:
- Resolve scheme compatibility problems[1].
- Remove the MD5 hashes in RC1
- Rename the source release file to 'dubbo-incubating-2.6.2-source-release'.
- Polish the Code Conduct,  Contributing Doc, some pom files to remove some out of date informations.
- Remove some unrelated log files and empty directories from the source release, try to keep source release the same with release tag.
- Temporarily close checkstyle-plugin by default.

To summary the vote result this round:
+1 (binding), Justin Mclean
-1 (binding), Willem Jiang 

The discussion thread in Dubbo community about cancel this one and start a new candidate:
https://lists.apache.org/thread.html/e871b5fc5bac3c588514e6cf57f3520b16c088ca1ed483f0ffe582fb@%3Cdev.dubbo.apache.org%3E

1. https://lists.apache.org/thread.html/96af7381da0fbc1dc0895134abc5f8faf892a72061929489082f5e6a@%3Cgeneral.incubator.apache.org%3E

Best regards,
Jun

> On 18 May 2018, at 5:26 PM, Jun Liu <li...@apache.org> wrote:
> 
> Hello All,
> 
> This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.
> 
> The Apache Dubbo community has voted on and approved a proposal to release Apache Dubbo (Incubating) version 2.6.2.
> 
> We now kindly request the Incubator PMC members review and vote on this incubator release.
> 
> Apache Dubbo™ (incubating) is a high-performance, java based, open source RPC framework. Dubbo offers three key functionalities, which include interface based remote call, fault tolerance & load balancing, and automatic service registration & discovery. 
> 
> Dubbo vote thread:
> https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd971bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E
> 
> Dubbo vote result thread:
> https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E
> 
> The release candidates:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2
> 
> Git tag for the release:
> https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2 
> 
> Hash for the release tag:
> afab04c53edab38d52275d2a198ea1aff7a4f41e
> 
> Release Notes:
> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md
> 
> The artifacts have been signed with Key : 28681CB1, which can be found in the keys file:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS
> 
> The vote will be open for at least 72 hours or until necessary number of votes are reached.
> 
> Please vote accordingly:
> [ ] +1 approve 
> [ ] +0 no opinion 
> [ ] -1 disapprove with the reason
> 
> Thanks.
> Jun Liu,
> The Apache Dubbo (Incubating) Team
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Ian Luo <ia...@gmail.com>.
We decide to withdraw this release vote, and redo this release. Not only
because we want  a better release which honors feedbacks especially from
William and Justin, but also because we need to address one compatibility
issue found recently [1].

Thank you all for your feedback, hopefully we can start the new round vote
today or tomorrow :)

Regards,
-Ian.

1. https://github.com/apache/incubator-dubbo/issues/1816: dubbo schema
compatiblity.

On Mon, May 21, 2018 at 4:00 PM Justin Mclean <ju...@classsoftware.com>
wrote:

> Hi,
>
> > As I cannot build the binary without doing some changes on the pom file,
> I
> > had to vote -1 (binding) here.
>
> You don’t need to modify the pom file to compile you can specify
> -Dcheckstyle.skip to compile id check style is acting up for you.
>
> Up to you if you want to vote -1 but I just want to remind people here
> (mostly the Dubbo PMC who are new to this) that:
> - a -1 vote is not a veto (but should be carefully considered)
> - a release only need 3 +1 votes and more +1 than -1 to pass
> - you can vote +0 or change you vote if you think the issue can be fixed
> next release
>
> > 2. I checked the source zip[1], it is not same with github tag one[2].
> > There are some log files and empty directories which is not in the git
> > repo, I guess it relates to the builder's workspace.
>
> Which is OK many project release are not 100% identical to what is in the
> release tar what is important is that none of the source code has changed.
>
> Thanks,
> Justin
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by sebb <se...@gmail.com>.
On 21 May 2018 at 21:16, Justin Mclean <ju...@classsoftware.com> wrote:
> Hi,
>
>> I don't think there should be anything in the source release that is
>> not in the source repo.
>
> It reasonably  common (from my review of 100’s of releases) that the build process may modify and/or add some files (e.g config files). I wouldn’t expect any code to change or be added.

It's also important that there is nothing in the release tar that is
not in the source repo or which cannot be readily derived from it
(though that is too broad an exception).

I have seen release tars that include spurious files that happened to
be present in the workspace.
In one case a file held private information.

Your original mail said:

>>> Which is OK many project release are not 100% identical to what is in the release tar what is important is that none of the source code has changed.

This could be taken to mean that it does not matter what goes into the
release tar other than the source files.

I think it's important to be clear that the release tar should have
minimal changes from the source repo, and what sort of changes are
allowed.

> Thanks,
> Justin

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

> I don't think there should be anything in the source release that is
> not in the source repo.

It reasonably  common (from my review of 100’s of releases) that the build process may modify and/or add some files (e.g config files). I wouldn’t expect any code to change or be added.

Thanks,
Justin

Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by sebb <se...@gmail.com>.
On 21 May 2018 at 08:59, Justin Mclean <ju...@classsoftware.com> wrote:
> Hi,
>
>> As I cannot build the binary without doing some changes on the pom file, I
>> had to vote -1 (binding) here.
>
> You don’t need to modify the pom file to compile you can specify -Dcheckstyle.skip to compile id check style is acting up for you.
>
> Up to you if you want to vote -1 but I just want to remind people here (mostly the Dubbo PMC who are new to this) that:
> - a -1 vote is not a veto (but should be carefully considered)
> - a release only need 3 +1 votes and more +1 than -1 to pass
> - you can vote +0 or change you vote if you think the issue can be fixed next release
>
>> 2. I checked the source zip[1], it is not same with github tag one[2].
>> There are some log files and empty directories which is not in the git
>> repo, I guess it relates to the builder's workspace.
>
> Which is OK many project release are not 100% identical to what is in the release tar what is important is that none of the source code has changed.

Huh?

I don't think there should be anything in the source release that is
not in the source repo.

> Thanks,
> Justin
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

> As I cannot build the binary without doing some changes on the pom file, I
> had to vote -1 (binding) here.

You don’t need to modify the pom file to compile you can specify -Dcheckstyle.skip to compile id check style is acting up for you.

Up to you if you want to vote -1 but I just want to remind people here (mostly the Dubbo PMC who are new to this) that:
- a -1 vote is not a veto (but should be carefully considered)
- a release only need 3 +1 votes and more +1 than -1 to pass
- you can vote +0 or change you vote if you think the issue can be fixed next release

> 2. I checked the source zip[1], it is not same with github tag one[2].
> There are some log files and empty directories which is not in the git
> repo, I guess it relates to the builder's workspace.

Which is OK many project release are not 100% identical to what is in the release tar what is important is that none of the source code has changed.

Thanks,
Justin


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Mark Thomas <ma...@apache.org>.
On 23/05/18 09:05, jun liu wrote:
>> Using "I think we should ..." rather than "We plan..." (or something
>> along those lines) would make it clearer that it was your
>> idea/suggestion rather than a community decision.
> 
> My apologies for the misleading and English fluency. 😂

No need to apologise. Your English is far better than my Chinese.

The reason I mentioned this particular case is that it is important that
it is clear that decisions are being made by the community, on the
mailing lists.

 > BTW, the discussion has started in Dubbo community:
> https://lists.apache.org/thread.html/e871b5fc5bac3c588514e6cf57f3520b16c088ca1ed483f0ffe582fb@%3Cdev.dubbo.apache.org%3E <https://lists.apache.org/thread.html/e871b5fc5bac3c588514e6cf57f3520b16c088ca1ed483f0ffe582fb@%3Cdev.dubbo.apache.org%3E>

I saw that. Looks good.

Thanks,

Mark


> 
> Best regards,
> Jun
> 
>> On 22 May 2018, at 4:45 PM, Mark Thomas <ma...@apache.org> wrote:
>>
>> On 22/05/18 02:53, Jun Liu wrote:
>>>> Where did the community discuss this? I'd expect to see a discussion on
>>>> a mailing list before a decision is announced.
>>>
>>> Haven’t discussed yet, i will start one now.
>>
>> Ah. OK. Just a phrasing issue then. No problem.
>>
>> Using "I think we should ..." rather than "We plan..." (or something
>> along those lines) would make it clearer that it was your
>> idea/suggestion rather than a community decision.
>>
>> Mark
>>
>>>
>>> Best regards,
>>> Jun
>>>
>>>> On 22 May 2018, at 2:15 AM, Mark Thomas <ma...@apache.org> wrote:
>>>>
>>>> On 21/05/18 07:32, Jun Liu wrote:
>>>>
>>>> <snip/>
>>>>
>>>>> we plan to solve this problem and start a new round of voting.
>>>>
>>>> Where did the community discuss this? I'd expect to see a discussion on
>>>> a mailing list before a decision is announced.
>>>>
>>>> Mark
>>>
>>
> 
> 


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by jun liu <ke...@gmail.com>.
> Using "I think we should ..." rather than "We plan..." (or something
> along those lines) would make it clearer that it was your
> idea/suggestion rather than a community decision.

My apologies for the misleading and English fluency. 😂

BTW, the discussion has started in Dubbo community:
https://lists.apache.org/thread.html/e871b5fc5bac3c588514e6cf57f3520b16c088ca1ed483f0ffe582fb@%3Cdev.dubbo.apache.org%3E <https://lists.apache.org/thread.html/e871b5fc5bac3c588514e6cf57f3520b16c088ca1ed483f0ffe582fb@%3Cdev.dubbo.apache.org%3E>

Best regards,
Jun

> On 22 May 2018, at 4:45 PM, Mark Thomas <ma...@apache.org> wrote:
> 
> On 22/05/18 02:53, Jun Liu wrote:
>>> Where did the community discuss this? I'd expect to see a discussion on
>>> a mailing list before a decision is announced.
>> 
>> Haven’t discussed yet, i will start one now.
> 
> Ah. OK. Just a phrasing issue then. No problem.
> 
> Using "I think we should ..." rather than "We plan..." (or something
> along those lines) would make it clearer that it was your
> idea/suggestion rather than a community decision.
> 
> Mark
> 
>> 
>> Best regards,
>> Jun
>> 
>>> On 22 May 2018, at 2:15 AM, Mark Thomas <ma...@apache.org> wrote:
>>> 
>>> On 21/05/18 07:32, Jun Liu wrote:
>>> 
>>> <snip/>
>>> 
>>>> we plan to solve this problem and start a new round of voting.
>>> 
>>> Where did the community discuss this? I'd expect to see a discussion on
>>> a mailing list before a decision is announced.
>>> 
>>> Mark
>> 
> 


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Mark Thomas <ma...@apache.org>.
On 22/05/18 02:53, Jun Liu wrote:
>> Where did the community discuss this? I'd expect to see a discussion on
>> a mailing list before a decision is announced.
> 
> Haven’t discussed yet, i will start one now.

Ah. OK. Just a phrasing issue then. No problem.

Using "I think we should ..." rather than "We plan..." (or something
along those lines) would make it clearer that it was your
idea/suggestion rather than a community decision.

Mark

> 
> Best regards,
> Jun
> 
>> On 22 May 2018, at 2:15 AM, Mark Thomas <ma...@apache.org> wrote:
>>
>> On 21/05/18 07:32, Jun Liu wrote:
>>
>> <snip/>
>>
>>> we plan to solve this problem and start a new round of voting.
>>
>> Where did the community discuss this? I'd expect to see a discussion on
>> a mailing list before a decision is announced.
>>
>> Mark
> 


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Jun Liu <li...@apache.org>.
> Where did the community discuss this? I'd expect to see a discussion on
> a mailing list before a decision is announced.

Haven’t discussed yet, i will start one now.

Best regards,
Jun

> On 22 May 2018, at 2:15 AM, Mark Thomas <ma...@apache.org> wrote:
> 
> On 21/05/18 07:32, Jun Liu wrote:
> 
> <snip/>
> 
>> we plan to solve this problem and start a new round of voting.
> 
> Where did the community discuss this? I'd expect to see a discussion on
> a mailing list before a decision is announced.
> 
> Mark


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Mark Thomas <ma...@apache.org>.
On 21/05/18 07:32, Jun Liu wrote:

<snip/>

> we plan to solve this problem and start a new round of voting.

Where did the community discuss this? I'd expect to see a discussion on
a mailing list before a decision is announced.

Mark

Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Jun Liu <li...@apache.org>.
> 1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
> maven release plugin doesn't take all of them.
> .//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
> .//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> 
> 2. I checked the source zip[1], it is not same with github tag one[2].
> There are some log files and empty directories which is not in the git
> repo, I guess it relates to the builder's workspace.
> It could be addressed by generate the source release kit in the
> distribution like we did here[3].

We rely on the release and assembly plugin to finish these works, there may have some inappropriate plugin configuration. I will check the problems you mentioned and fix them by following your guides.

> 3. I tried to build the source from kit, it looks the check style is
> enabled when since JDK 1.8 and I just got the out of memory error there.
> I found there are comment about this check style issue (it takes lots of
> time and it doesn't work on my box). So I created an issue[4] of it.
> I can build the source by modify the pom to skip the check style check, but
> it kind of hiking , so I had to vote -1 (binding) for it.

During the internal dubbo community vote, Justin, Huxing and Ian also reported the checkstyle plugin problem (cost time and resources) when build. Since this problem has confused you and maybe others from building the source, we plan to solve this problem and start a new round of voting.
 
> 4. I checked the binary kit, it only includes the jars file of dubbo[5], as
> we cannot provide the apache maven release now, it could cause some trouble
> for the user to consume the binary without third party dependencies. Do we
> have any plan for it ?

Besides the apache binary release, we will also publish the convenient maven artifacts to the central maven repository, which are not mentioned in this vote thread since it’s not a necessary part of the apache release. This way, users can consume from the central repository as before. 
In the next release, when the groupid change to “org.apache.dubbo”, we will switch from central maven repository to apache maven repository, and will include the apache maven staging repo address in the vote. The current binary zip, as you point out "only includes the jars file of dubbo[5]”, will be removed or replaced with more meaningful bin release, such as samples or quick start guides which users can help users start the journey of dubbo quickly.

To conclude, we decide to start a new round of voting regarding the problems found during this round. Thanks for the vote.

Best regards,
Jun

> On 21 May 2018, at 10:34 AM, Willem Jiang <wi...@gmail.com> wrote:
> 
> Hi,
> 
> I did some check on the source zip and binary zip, the License and Notice
> files looks good. But I also found some issues here.
> As I cannot build the binary without doing some changes on the pom file, I
> had to vote -1 (binding) here.
> 
> Here are the issues that I found:
> 
> 1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
> maven release plugin doesn't take all of them.
> .//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
> .//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> 
> 2. I checked the source zip[1], it is not same with github tag one[2].
> There are some log files and empty directories which is not in the git
> repo, I guess it relates to the builder's workspace.
> It could be addressed by generate the source release kit in the
> distribution like we did here[3].
> BTW, it's quite important that every one can build the kit by following the
> instruction. I didn't found the instruction file from the README.
> 
> 3. I tried to build the source from kit, it looks the check style is
> enabled when since JDK 1.8 and I just got the out of memory error there.
> I found there are comment about this check style issue (it takes lots of
> time and it doesn't work on my box). So I created an issue[4] of it.
> I can build the source by modify the pom to skip the check style check, but
> it kind of hiking , so I had to vote -1 (binding) for it.
> 
> 4. I checked the binary kit, it only includes the jars file of dubbo[5], as
> we cannot provide the apache maven release now, it could cause some trouble
> for the user to consume the binary without third party dependencies. Do we
> have any plan for it ?
> 
> Another tip for the release check is running some applications which is
> build on the top of the framework to make sure all the distribution is good.
> 
> 
> [1]
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2/dubbo-incubating-2.6.2-source-release.zip
> [2]https://github.com/apache/incubator-dubbo/archive/dubbo-2.6.2.zip
> [3]
> https://github.com/apache/incubator-servicecomb-saga/blob/master/saga-distribution/src/assembly/src.xml
> [4]https://github.com/apache/incubator-dubbo/issues/1821
> [5]
> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/distribution/src/assembly/release-bin.xml#L43
> 
> 
> 
> Willem Jiang
> 
> Blog: http://willemjiang.blogspot.com (English)
>          http://jnn.iteye.com  (Chinese)
> Twitter: willemjiang
> Weibo: 姜宁willem
> 
> On Fri, May 18, 2018 at 5:26 PM, Jun Liu <li...@apache.org> wrote:
> 
>> Hello All,
>> 
>> This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.
>> 
>> The Apache Dubbo community has voted on and approved a proposal to release
>> Apache Dubbo (Incubating) version 2.6.2.
>> 
>> We now kindly request the Incubator PMC members review and vote on this
>> incubator release.
>> 
>> Apache Dubbo™ (incubating) is a high-performance, java based, open source
>> RPC framework. Dubbo offers three key functionalities, which include
>> interface based remote call, fault tolerance & load balancing, and
>> automatic service registration & discovery.
>> 
>> Dubbo vote thread:
>> https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd9
>> 71bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E
>> 
>> Dubbo vote result thread:
>> https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3
>> 826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E
>> 
>> The release candidates:
>> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2
>> 
>> Git tag for the release:
>> https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2
>> 
>> Hash for the release tag:
>> afab04c53edab38d52275d2a198ea1aff7a4f41e
>> 
>> Release Notes:
>> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md
>> 
>> The artifacts have been signed with Key : 28681CB1, which can be found in
>> the keys file:
>> https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS
>> 
>> The vote will be open for at least 72 hours or until necessary number of
>> votes are reached.
>> 
>> Please vote accordingly:
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove with the reason
>> 
>> Thanks.
>> Jun Liu,
>> The Apache Dubbo (Incubating) Team
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: general-help@incubator.apache.org
>> 
>> 


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Jun Liu <li...@apache.org>.
> 1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
> maven release plugin doesn't take all of them.
> .//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
> .//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> 
> 2. I checked the source zip[1], it is not same with github tag one[2].
> There are some log files and empty directories which is not in the git
> repo, I guess it relates to the builder's workspace.
> It could be addressed by generate the source release kit in the
> distribution like we did here[3].

We rely on the release and assembly plugin to finish these works, there may have some inappropriate plugin configuration. I will check the problems you mentioned and fix them by following your guides.

> 3. I tried to build the source from kit, it looks the check style is
> enabled when since JDK 1.8 and I just got the out of memory error there.
> I found there are comment about this check style issue (it takes lots of
> time and it doesn't work on my box). So I created an issue[4] of it.
> I can build the source by modify the pom to skip the check style check, but
> it kind of hiking , so I had to vote -1 (binding) for it.

During the internal dubbo community vote, Justin, Huxing and Ian also reported the checkstyle plugin problem (cost time and resources) when build. Since this problem has confused you and maybe others from building the source, we plan to solve this problem and start a new round of voting.
 
> 4. I checked the binary kit, it only includes the jars file of dubbo[5], as
> we cannot provide the apache maven release now, it could cause some trouble
> for the user to consume the binary without third party dependencies. Do we
> have any plan for it ?

Besides the apache binary release, we will also publish the convenient maven artifacts to the central maven repository, which are not mentioned in this vote thread since it’s not a necessary part of the apache release. This way, users can consume from the central repository as before. 
In the next release, when the groupid change to “org.apache.dubbo”, we will switch from central maven repository to apache maven repository, and will include the apache maven staging repo address in the vote. The current binary zip, as you point out "only includes the jars file of dubbo[5]”, will be removed or replaced with more meaningful bin release, such as samples or quick start guides which users can help users start the journey of dubbo quickly.

To conclude, we decide to start a new round of voting regarding the problems found during this round. Thanks for the vote.

Best regards,
Jun

> On 21 May 2018, at 10:34 AM, Willem Jiang <wi...@gmail.com> wrote:
> 
> Hi,
> 
> I did some check on the source zip and binary zip, the License and Notice
> files looks good. But I also found some issues here.
> As I cannot build the binary without doing some changes on the pom file, I
> had to vote -1 (binding) here.
> 
> Here are the issues that I found:
> 
> 1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
> maven release plugin doesn't take all of them.
> .//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> .//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
> .//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
> 
> 2. I checked the source zip[1], it is not same with github tag one[2].
> There are some log files and empty directories which is not in the git
> repo, I guess it relates to the builder's workspace.
> It could be addressed by generate the source release kit in the
> distribution like we did here[3].
> BTW, it's quite important that every one can build the kit by following the
> instruction. I didn't found the instruction file from the README.
> 
> 3. I tried to build the source from kit, it looks the check style is
> enabled when since JDK 1.8 and I just got the out of memory error there.
> I found there are comment about this check style issue (it takes lots of
> time and it doesn't work on my box). So I created an issue[4] of it.
> I can build the source by modify the pom to skip the check style check, but
> it kind of hiking , so I had to vote -1 (binding) for it.
> 
> 4. I checked the binary kit, it only includes the jars file of dubbo[5], as
> we cannot provide the apache maven release now, it could cause some trouble
> for the user to consume the binary without third party dependencies. Do we
> have any plan for it ?
> 
> Another tip for the release check is running some applications which is
> build on the top of the framework to make sure all the distribution is good.
> 
> 
> [1]
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2/dubbo-incubating-2.6.2-source-release.zip
> [2]https://github.com/apache/incubator-dubbo/archive/dubbo-2.6.2.zip
> [3]
> https://github.com/apache/incubator-servicecomb-saga/blob/master/saga-distribution/src/assembly/src.xml
> [4]https://github.com/apache/incubator-dubbo/issues/1821
> [5]
> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/distribution/src/assembly/release-bin.xml#L43
> 
> 
> 
> Willem Jiang
> 
> Blog: http://willemjiang.blogspot.com (English)
>          http://jnn.iteye.com  (Chinese)
> Twitter: willemjiang
> Weibo: 姜宁willem
> 
> On Fri, May 18, 2018 at 5:26 PM, Jun Liu <li...@apache.org> wrote:
> 
>> Hello All,
>> 
>> This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.
>> 
>> The Apache Dubbo community has voted on and approved a proposal to release
>> Apache Dubbo (Incubating) version 2.6.2.
>> 
>> We now kindly request the Incubator PMC members review and vote on this
>> incubator release.
>> 
>> Apache Dubbo™ (incubating) is a high-performance, java based, open source
>> RPC framework. Dubbo offers three key functionalities, which include
>> interface based remote call, fault tolerance & load balancing, and
>> automatic service registration & discovery.
>> 
>> Dubbo vote thread:
>> https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd9
>> 71bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E
>> 
>> Dubbo vote result thread:
>> https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3
>> 826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E
>> 
>> The release candidates:
>> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2
>> 
>> Git tag for the release:
>> https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2
>> 
>> Hash for the release tag:
>> afab04c53edab38d52275d2a198ea1aff7a4f41e
>> 
>> Release Notes:
>> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md
>> 
>> The artifacts have been signed with Key : 28681CB1, which can be found in
>> the keys file:
>> https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS
>> 
>> The vote will be open for at least 72 hours or until necessary number of
>> votes are reached.
>> 
>> Please vote accordingly:
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove with the reason
>> 
>> Thanks.
>> Jun Liu,
>> The Apache Dubbo (Incubating) Team
>> 
>> 
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: general-help@incubator.apache.org
>> 
>> 


---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Willem Jiang <wi...@gmail.com>.
Hi,

I did some check on the source zip and binary zip, the License and Notice
files looks good. But I also found some issues here.
As I cannot build the binary without doing some changes on the pom file, I
had to vote -1 (binding) here.

Here are the issues that I found:

1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
maven release plugin doesn't take all of them.
.//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
.//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
.//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
.//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>

2. I checked the source zip[1], it is not same with github tag one[2].
There are some log files and empty directories which is not in the git
repo, I guess it relates to the builder's workspace.
It could be addressed by generate the source release kit in the
distribution like we did here[3].
BTW, it's quite important that every one can build the kit by following the
instruction. I didn't found the instruction file from the README.

3. I tried to build the source from kit, it looks the check style is
enabled when since JDK 1.8 and I just got the out of memory error there.
I found there are comment about this check style issue (it takes lots of
time and it doesn't work on my box). So I created an issue[4] of it.
I can build the source by modify the pom to skip the check style check, but
it kind of hiking , so I had to vote -1 (binding) for it.

4. I checked the binary kit, it only includes the jars file of dubbo[5], as
we cannot provide the apache maven release now, it could cause some trouble
for the user to consume the binary without third party dependencies. Do we
have any plan for it ?

Another tip for the release check is running some applications which is
build on the top of the framework to make sure all the distribution is good.


[1]
https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2/dubbo-incubating-2.6.2-source-release.zip
[2]https://github.com/apache/incubator-dubbo/archive/dubbo-2.6.2.zip
[3]
https://github.com/apache/incubator-servicecomb-saga/blob/master/saga-distribution/src/assembly/src.xml
[4]https://github.com/apache/incubator-dubbo/issues/1821
[5]
https://github.com/apache/incubator-dubbo/blob/2.6.2-release/distribution/src/assembly/release-bin.xml#L43



Willem Jiang

Blog: http://willemjiang.blogspot.com (English)
          http://jnn.iteye.com  (Chinese)
Twitter: willemjiang
Weibo: 姜宁willem

On Fri, May 18, 2018 at 5:26 PM, Jun Liu <li...@apache.org> wrote:

> Hello All,
>
> This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.
>
> The Apache Dubbo community has voted on and approved a proposal to release
> Apache Dubbo (Incubating) version 2.6.2.
>
> We now kindly request the Incubator PMC members review and vote on this
> incubator release.
>
> Apache Dubbo™ (incubating) is a high-performance, java based, open source
> RPC framework. Dubbo offers three key functionalities, which include
> interface based remote call, fault tolerance & load balancing, and
> automatic service registration & discovery.
>
> Dubbo vote thread:
> https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd9
> 71bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E
>
> Dubbo vote result thread:
> https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3
> 826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E
>
> The release candidates:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2
>
> Git tag for the release:
> https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2
>
> Hash for the release tag:
> afab04c53edab38d52275d2a198ea1aff7a4f41e
>
> Release Notes:
> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md
>
> The artifacts have been signed with Key : 28681CB1, which can be found in
> the keys file:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS
>
> The vote will be open for at least 72 hours or until necessary number of
> votes are reached.
>
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thanks.
> Jun Liu,
> The Apache Dubbo (Incubating) Team
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>

Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,

+1 (binding)

Carrying my vote over form the dev list and all the usual checks done.

 There’s a few very minor / trivial documentation issues that need fixed but hashes, signatures, license, notice etc etc is all good.

Thanks,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org


Re: [VOTE]: Release Apache Dubbo (Incubating) 2.6.2 [RC1]

Posted by Willem Jiang <wi...@gmail.com>.
Hi,

I did some check on the source zip and binary zip, the License and Notice
files looks good. But I also found some issues here.
As I cannot build the binary without doing some changes on the pom file, I
had to vote -1 (binding) here.

Here are the issues that I found:

1. there is bunch of dubbo-2.6.2-SNAPSHOT  in the pom.xml file, I guess the
maven release plugin doesn't take all of them.
.//bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
.//dependencies-bom/pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>
.//distribution/pom.xml:        <version>2.6.2-SNAPSHOT-SNAPSHOT</version>
.//pom.xml:      <tag>dubbo-2.6.2-SNAPSHOT</tag>

2. I checked the source zip[1], it is not same with github tag one[2].
There are some log files and empty directories which is not in the git
repo, I guess it relates to the builder's workspace.
It could be addressed by generate the source release kit in the
distribution like we did here[3].
BTW, it's quite important that every one can build the kit by following the
instruction. I didn't found the instruction file from the README.

3. I tried to build the source from kit, it looks the check style is
enabled when since JDK 1.8 and I just got the out of memory error there.
I found there are comment about this check style issue (it takes lots of
time and it doesn't work on my box). So I created an issue[4] of it.
I can build the source by modify the pom to skip the check style check, but
it kind of hiking , so I had to vote -1 (binding) for it.

4. I checked the binary kit, it only includes the jars file of dubbo[5], as
we cannot provide the apache maven release now, it could cause some trouble
for the user to consume the binary without third party dependencies. Do we
have any plan for it ?

Another tip for the release check is running some applications which is
build on the top of the framework to make sure all the distribution is good.


[1]
https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2/dubbo-incubating-2.6.2-source-release.zip
[2]https://github.com/apache/incubator-dubbo/archive/dubbo-2.6.2.zip
[3]
https://github.com/apache/incubator-servicecomb-saga/blob/master/saga-distribution/src/assembly/src.xml
[4]https://github.com/apache/incubator-dubbo/issues/1821
[5]
https://github.com/apache/incubator-dubbo/blob/2.6.2-release/distribution/src/assembly/release-bin.xml#L43



Willem Jiang

Blog: http://willemjiang.blogspot.com (English)
          http://jnn.iteye.com  (Chinese)
Twitter: willemjiang
Weibo: 姜宁willem

On Fri, May 18, 2018 at 5:26 PM, Jun Liu <li...@apache.org> wrote:

> Hello All,
>
> This is a call for vote to release Apache Dubbo (Incubating) version 2.6.2.
>
> The Apache Dubbo community has voted on and approved a proposal to release
> Apache Dubbo (Incubating) version 2.6.2.
>
> We now kindly request the Incubator PMC members review and vote on this
> incubator release.
>
> Apache Dubbo™ (incubating) is a high-performance, java based, open source
> RPC framework. Dubbo offers three key functionalities, which include
> interface based remote call, fault tolerance & load balancing, and
> automatic service registration & discovery.
>
> Dubbo vote thread:
> https://lists.apache.org/thread.html/2292dfa6698c97550419ca547c8cd9
> 71bae8c88c440ef277314c841f@%3Cdev.dubbo.apache.org%3E
>
> Dubbo vote result thread:
> https://lists.apache.org/thread.html/701b3d436920eea3030f2a977eb1d3
> 826774bc96e0b1d8b1d09a2ce8@%3Cdev.dubbo.apache.org%3E
>
> The release candidates:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/2.6.2
>
> Git tag for the release:
> https://github.com/apache/incubator-dubbo/tree/dubbo-2.6.2
>
> Hash for the release tag:
> afab04c53edab38d52275d2a198ea1aff7a4f41e
>
> Release Notes:
> https://github.com/apache/incubator-dubbo/blob/2.6.2-release/CHANGES.md
>
> The artifacts have been signed with Key : 28681CB1, which can be found in
> the keys file:
> https://dist.apache.org/repos/dist/dev/incubator/dubbo/KEYS
>
> The vote will be open for at least 72 hours or until necessary number of
> votes are reached.
>
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thanks.
> Jun Liu,
> The Apache Dubbo (Incubating) Team
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>