Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
Thanks John for the suggestion, I will add it into our release document. Will send out the vote result soon. 2015-11-11 9:11 GMT+08:00 John D. Ament : > +1 binding > > A few notes: > > - Reviewed contents, licenses all look correct. > - hashes line up. > - I'd recommend that the commit you use to vote on is the one that includes > the tagging, so > > https://github.com/apache/incubator-kylin/commit/6a9499b4d0fabb54211a8a536c2e18d3fe8b4a5d > should > probably be voted on, as its a little confusing to look at it otherwise. > > John > > On Sat, Nov 7, 2015 at 7:46 PM ShaoFeng Shi > wrote: > > > Hi all, > > > > The Apache Kylin community has voted on and approved a proposal to > release > > Apache Kylin 1.1.1-incubating. This release only includes license > cleanup, > > no Java code change; For more information please check KYLIN-999. > > > > Proposal: > > http://s.apache.org/kylin-1.1.1-vote_rc1 > > > > Vote result: > > 7 binding +1 votes > > 9 non-binding +1 votes > > No -1 votes > > http://s.apache.org/kylin-1.1.1-result_rc1 > > > > > > The commit to be voted upon: > > > > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > > > The artifacts to be voted on are located here: > > > > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > > > The hashes of the artifacts are as follows: > > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > > ff942b2eba870b04552bc8c8dace5517 > > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > > cc8990ba7821b02f4bf6da5f870b30a915881535 > > > > A staged Maven repository is available for review at: > > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > > > Release artifacts are signed with the following key: > > https://people.apache.org/keys/committer/shaofengshi.asc > > > > Pursuant to the Releases section of the Incubation Policy and with > > the endorsement of our mentors we would now like to request > > the permission of the Incubator PMC to publish the release. The vote > > is open for 72 hours, or until the necessary number of votes (3 +1) > > is reached. > > > > [ ] +1 Release this package > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > [ ] -1 Do not release this package because... > > > > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > > shaofeng...@apache.org > > > -- Best regards, Shaofeng Shi
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
+1 binding A few notes: - Reviewed contents, licenses all look correct. - hashes line up. - I'd recommend that the commit you use to vote on is the one that includes the tagging, so https://github.com/apache/incubator-kylin/commit/6a9499b4d0fabb54211a8a536c2e18d3fe8b4a5d should probably be voted on, as its a little confusing to look at it otherwise. John On Sat, Nov 7, 2015 at 7:46 PM ShaoFeng Shi wrote: > Hi all, > > The Apache Kylin community has voted on and approved a proposal to release > Apache Kylin 1.1.1-incubating. This release only includes license cleanup, > no Java code change; For more information please check KYLIN-999. > > Proposal: > http://s.apache.org/kylin-1.1.1-vote_rc1 > > Vote result: > 7 binding +1 votes > 9 non-binding +1 votes > No -1 votes > http://s.apache.org/kylin-1.1.1-result_rc1 > > > The commit to be voted upon: > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > The artifacts to be voted on are located here: > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > The hashes of the artifacts are as follows: > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > ff942b2eba870b04552bc8c8dace5517 > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > cc8990ba7821b02f4bf6da5f870b30a915881535 > > A staged Maven repository is available for review at: > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > Release artifacts are signed with the following key: > https://people.apache.org/keys/committer/shaofengshi.asc > > Pursuant to the Releases section of the Incubation Policy and with > the endorsement of our mentors we would now like to request > the permission of the Incubator PMC to publish the release. The vote > is open for 72 hours, or until the necessary number of votes (3 +1) > is reached. > > [ ] +1 Release this package > [ ] 0 I don't feel strongly about it, but I'm okay with the release > [ ] -1 Do not release this package because... > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > shaofeng...@apache.org >
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
+1 non-binding md5&sha1 passed no license issue mvn test passed On Tue, Nov 10, 2015 at 10:57 AM, Luke Han wrote: > Forwarding my vote from dev@: > > +1 non-binding > > > Best Regards! > - > > Luke Han > > On Tue, Nov 10, 2015 at 6:00 AM, Henry Saputra > wrote: > > > - Signature files look good > > - Hash files look good > > - LICENSE file looks good > > - NOTICE file looks good > > - DISCLAIMER files exists > > - Source compiled successfully > > - NO 3rd party executables in source artifact > > - Source files has correct Apache header > > > > +1 (binding) > > > > On Sat, Nov 7, 2015 at 4:46 PM, ShaoFeng Shi > > wrote: > > > Hi all, > > > > > > The Apache Kylin community has voted on and approved a proposal to > > release > > > Apache Kylin 1.1.1-incubating. This release only includes license > > cleanup, > > > no Java code change; For more information please check KYLIN-999. > > > > > > Proposal: > > > http://s.apache.org/kylin-1.1.1-vote_rc1 > > > > > > Vote result: > > > 7 binding +1 votes > > > 9 non-binding +1 votes > > > No -1 votes > > > http://s.apache.org/kylin-1.1.1-result_rc1 > > > > > > > > > The commit to be voted upon: > > > > > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > > > > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > > > > > The artifacts to be voted on are located here: > > > > > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > > > > > The hashes of the artifacts are as follows: > > > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > > > ff942b2eba870b04552bc8c8dace5517 > > > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > > > cc8990ba7821b02f4bf6da5f870b30a915881535 > > > > > > A staged Maven repository is available for review at: > > > > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > > > > > Release artifacts are signed with the following key: > > > https://people.apache.org/keys/committer/shaofengshi.asc > > > > > > Pursuant to the Releases section of the Incubation Policy and with > > > the endorsement of our mentors we would now like to request > > > the permission of the Incubator PMC to publish the release. The vote > > > is open for 72 hours, or until the necessary number of votes (3 +1) > > > is reached. > > > > > > [ ] +1 Release this package > > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > > [ ] -1 Do not release this package because... > > > > > > > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > > > shaofeng...@apache.org > > > > - > > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > For additional commands, e-mail: general-h...@incubator.apache.org > > > > >
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
Forwarding my vote from dev@: +1 non-binding Best Regards! - Luke Han On Tue, Nov 10, 2015 at 6:00 AM, Henry Saputra wrote: > - Signature files look good > - Hash files look good > - LICENSE file looks good > - NOTICE file looks good > - DISCLAIMER files exists > - Source compiled successfully > - NO 3rd party executables in source artifact > - Source files has correct Apache header > > +1 (binding) > > On Sat, Nov 7, 2015 at 4:46 PM, ShaoFeng Shi > wrote: > > Hi all, > > > > The Apache Kylin community has voted on and approved a proposal to > release > > Apache Kylin 1.1.1-incubating. This release only includes license > cleanup, > > no Java code change; For more information please check KYLIN-999. > > > > Proposal: > > http://s.apache.org/kylin-1.1.1-vote_rc1 > > > > Vote result: > > 7 binding +1 votes > > 9 non-binding +1 votes > > No -1 votes > > http://s.apache.org/kylin-1.1.1-result_rc1 > > > > > > The commit to be voted upon: > > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > > > The artifacts to be voted on are located here: > > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > > > The hashes of the artifacts are as follows: > > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > > ff942b2eba870b04552bc8c8dace5517 > > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > > cc8990ba7821b02f4bf6da5f870b30a915881535 > > > > A staged Maven repository is available for review at: > > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > > > Release artifacts are signed with the following key: > > https://people.apache.org/keys/committer/shaofengshi.asc > > > > Pursuant to the Releases section of the Incubation Policy and with > > the endorsement of our mentors we would now like to request > > the permission of the Incubator PMC to publish the release. The vote > > is open for 72 hours, or until the necessary number of votes (3 +1) > > is reached. > > > > [ ] +1 Release this package > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > [ ] -1 Do not release this package because... > > > > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > > shaofeng...@apache.org > > - > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
- Signature files look good - Hash files look good - LICENSE file looks good - NOTICE file looks good - DISCLAIMER files exists - Source compiled successfully - NO 3rd party executables in source artifact - Source files has correct Apache header +1 (binding) On Sat, Nov 7, 2015 at 4:46 PM, ShaoFeng Shi wrote: > Hi all, > > The Apache Kylin community has voted on and approved a proposal to release > Apache Kylin 1.1.1-incubating. This release only includes license cleanup, > no Java code change; For more information please check KYLIN-999. > > Proposal: > http://s.apache.org/kylin-1.1.1-vote_rc1 > > Vote result: > 7 binding +1 votes > 9 non-binding +1 votes > No -1 votes > http://s.apache.org/kylin-1.1.1-result_rc1 > > > The commit to be voted upon: > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > The artifacts to be voted on are located here: > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > The hashes of the artifacts are as follows: > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > ff942b2eba870b04552bc8c8dace5517 > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > cc8990ba7821b02f4bf6da5f870b30a915881535 > > A staged Maven repository is available for review at: > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > Release artifacts are signed with the following key: > https://people.apache.org/keys/committer/shaofengshi.asc > > Pursuant to the Releases section of the Incubation Policy and with > the endorsement of our mentors we would now like to request > the permission of the Incubator PMC to publish the release. The vote > is open for 72 hours, or until the necessary number of votes (3 +1) > is reached. > > [ ] +1 Release this package > [ ] 0 I don't feel strongly about it, but I'm okay with the release > [ ] -1 Do not release this package because... > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > shaofeng...@apache.org - To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
Forwarding my vote from dev@: +1 binding Julian - To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
HI, +1 binding I checked: - artefact has incubating in name - DISCLAIMER exists - LICENSE and NOTICE correct - No unexpected binaries in source - All source file have Apache headers - Can compile from source Thanks, Justin - To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
Thanks Luke for answering this for me; yes the commit (619499b) is automatically made by the maven, and I picked the last commit made by human as the base to vote; Just let me know if this is an issue and I can send an updated version soon. Thanks! 2015-11-09 8:33 GMT+08:00 Luke Han : > Hi John, > The commit (619499b) you mentioned is auto generated by maven release > tool during script execution which changed the version to remove SNAPSHOT > just followed latest commit: 708ea3 (by committer). > The main change of this release is to totally remove Google Font, > please refer to KYLIN-999 for more detail [1] and v1.1.1-incubating commit > history [2] > > > > Thanks. > > > [1]. https://issues.apache.org/jira/browse/KYLIN-999 > [2]. > https://github.com/apache/incubator-kylin/commits/kylin-1.1.1-incubating > > > > > Best Regards! > - > > Luke Han > > On Mon, Nov 9, 2015 at 12:23 AM, John D. Ament > wrote: > > > It's a bit confusing, because I would have expected the vote to be based > on > > commit > > > > > https://github.com/apache/incubator-kylin/commit/6a9499b4d0fabb54211a8a536c2e18d3fe8b4a5d > > > > If you follow the commit above, then only KYLIN-999 is included in this > > release (over 1.1 release). Whereas if you follow 708ea3, much more > > content is included. Can you clarify which commit is accurate? > > > > John > > > > On Sat, Nov 7, 2015 at 7:46 PM ShaoFeng Shi > > wrote: > > > > > Hi all, > > > > > > The Apache Kylin community has voted on and approved a proposal to > > release > > > Apache Kylin 1.1.1-incubating. This release only includes license > > cleanup, > > > no Java code change; For more information please check KYLIN-999. > > > > > > Proposal: > > > http://s.apache.org/kylin-1.1.1-vote_rc1 > > > > > > Vote result: > > > 7 binding +1 votes > > > 9 non-binding +1 votes > > > No -1 votes > > > http://s.apache.org/kylin-1.1.1-result_rc1 > > > > > > > > > The commit to be voted upon: > > > > > > > > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > > > > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > > > > > The artifacts to be voted on are located here: > > > > > > > > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > > > > > The hashes of the artifacts are as follows: > > > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > > > ff942b2eba870b04552bc8c8dace5517 > > > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > > > cc8990ba7821b02f4bf6da5f870b30a915881535 > > > > > > A staged Maven repository is available for review at: > > > > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > > > > > Release artifacts are signed with the following key: > > > https://people.apache.org/keys/committer/shaofengshi.asc > > > > > > Pursuant to the Releases section of the Incubation Policy and with > > > the endorsement of our mentors we would now like to request > > > the permission of the Incubator PMC to publish the release. The vote > > > is open for 72 hours, or until the necessary number of votes (3 +1) > > > is reached. > > > > > > [ ] +1 Release this package > > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > > [ ] -1 Do not release this package because... > > > > > > > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > > > shaofeng...@apache.org > > > > > > -- Best regards, Shaofeng Shi
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
Hi John, The commit (619499b) you mentioned is auto generated by maven release tool during script execution which changed the version to remove SNAPSHOT just followed latest commit: 708ea3 (by committer). The main change of this release is to totally remove Google Font, please refer to KYLIN-999 for more detail [1] and v1.1.1-incubating commit history [2] Thanks. [1]. https://issues.apache.org/jira/browse/KYLIN-999 [2]. https://github.com/apache/incubator-kylin/commits/kylin-1.1.1-incubating Best Regards! - Luke Han On Mon, Nov 9, 2015 at 12:23 AM, John D. Ament wrote: > It's a bit confusing, because I would have expected the vote to be based on > commit > > https://github.com/apache/incubator-kylin/commit/6a9499b4d0fabb54211a8a536c2e18d3fe8b4a5d > > If you follow the commit above, then only KYLIN-999 is included in this > release (over 1.1 release). Whereas if you follow 708ea3, much more > content is included. Can you clarify which commit is accurate? > > John > > On Sat, Nov 7, 2015 at 7:46 PM ShaoFeng Shi > wrote: > > > Hi all, > > > > The Apache Kylin community has voted on and approved a proposal to > release > > Apache Kylin 1.1.1-incubating. This release only includes license > cleanup, > > no Java code change; For more information please check KYLIN-999. > > > > Proposal: > > http://s.apache.org/kylin-1.1.1-vote_rc1 > > > > Vote result: > > 7 binding +1 votes > > 9 non-binding +1 votes > > No -1 votes > > http://s.apache.org/kylin-1.1.1-result_rc1 > > > > > > The commit to be voted upon: > > > > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > > > The artifacts to be voted on are located here: > > > > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > > > The hashes of the artifacts are as follows: > > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > > ff942b2eba870b04552bc8c8dace5517 > > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > > cc8990ba7821b02f4bf6da5f870b30a915881535 > > > > A staged Maven repository is available for review at: > > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > > > Release artifacts are signed with the following key: > > https://people.apache.org/keys/committer/shaofengshi.asc > > > > Pursuant to the Releases section of the Incubation Policy and with > > the endorsement of our mentors we would now like to request > > the permission of the Incubator PMC to publish the release. The vote > > is open for 72 hours, or until the necessary number of votes (3 +1) > > is reached. > > > > [ ] +1 Release this package > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > [ ] -1 Do not release this package because... > > > > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > > shaofeng...@apache.org > > >
Re: [VOTE] Release Apache Kylin 1.1.1 (incubating)
It's a bit confusing, because I would have expected the vote to be based on commit https://github.com/apache/incubator-kylin/commit/6a9499b4d0fabb54211a8a536c2e18d3fe8b4a5d If you follow the commit above, then only KYLIN-999 is included in this release (over 1.1 release). Whereas if you follow 708ea3, much more content is included. Can you clarify which commit is accurate? John On Sat, Nov 7, 2015 at 7:46 PM ShaoFeng Shi wrote: > Hi all, > > The Apache Kylin community has voted on and approved a proposal to release > Apache Kylin 1.1.1-incubating. This release only includes license cleanup, > no Java code change; For more information please check KYLIN-999. > > Proposal: > http://s.apache.org/kylin-1.1.1-vote_rc1 > > Vote result: > 7 binding +1 votes > 9 non-binding +1 votes > No -1 votes > http://s.apache.org/kylin-1.1.1-result_rc1 > > > The commit to be voted upon: > > https://github.com/apache/incubator-kylin/commit/708ea38d7f764048e67e8e1ee8a13955983d12ba > > Its hash is 708ea38d7f764048e67e8e1ee8a13955983d12ba. > > The artifacts to be voted on are located here: > > https://dist.apache.org/repos/dist/dev/incubator/kylin/apache-kylin-1.1.1-incubating-rc1/ > > The hashes of the artifacts are as follows: > apache-kylin-1.1.1-incubating-src.tar.gz.md5 > ff942b2eba870b04552bc8c8dace5517 > apache-kylin-1.1.1-incubating-src.tar.gz.sha1 > cc8990ba7821b02f4bf6da5f870b30a915881535 > > A staged Maven repository is available for review at: > https://repository.apache.org/content/repositories/orgapachekylin-1013/ > > Release artifacts are signed with the following key: > https://people.apache.org/keys/committer/shaofengshi.asc > > Pursuant to the Releases section of the Incubation Policy and with > the endorsement of our mentors we would now like to request > the permission of the Incubator PMC to publish the release. The vote > is open for 72 hours, or until the necessary number of votes (3 +1) > is reached. > > [ ] +1 Release this package > [ ] 0 I don't feel strongly about it, but I'm okay with the release > [ ] -1 Do not release this package because... > > > Shaofeng Shi, on behalf of Apache Kylin PPMC > shaofeng...@apache.org >
