+1 (non-binding) - downloaded the artifacts, checked hash, verified signature, built and ran tests. All are OK - use `diff -r` to compare the artifact and the git tag. Like what Julian has pointed out, I can find a difference about license but it can be addressed after this vote.
-Rui On Tue, Jul 21, 2020 at 6:00 AM Ruben Q L <rube...@gmail.com> wrote: > Hello everyone, > > Thanks Chunwei for taking care of this release. > > +1 (non binding) > - Local Calcite build with tests (Windows10 + JDK8): OK > - Calcite-based application test suite: OK > > Similarly to Enrico, I also found some issues with the deprecated version > of the "standard" rules. > Initially I had a NPE [1] and also one test that was not working as > expected. I did not look much into it, but it seemed the same issue that > Enrico described. > All these problems were solved when I replaced the deprecated rules with > their new equivalents. > > Best, > Ruben > > [1] > Caused by: java.lang.NullPointerException: at index 0 > at > > com.onwbp.com.google.common.collect.ObjectArrays.checkElementNotNull(ObjectArrays.java:239) > at > > com.onwbp.com.google.common.collect.ObjectArrays.checkElementsNotNull(ObjectArrays.java:230) > at > > com.onwbp.com.google.common.collect.ObjectArrays.checkElementsNotNull(ObjectArrays.java:225) > at > > com.onwbp.com.google.common.collect.ImmutableList.construct(ImmutableList.java:281) > at > > com.onwbp.com.google.common.collect.ImmutableList.copyOf(ImmutableList.java:239) > at > > com.onwbp.com.google.common.collect.ImmutableList.copyOf(ImmutableList.java:209) > at com.onwbp.org.apache.calcite.tools.RuleSets.ofList(RuleSets.java:41) > > > Le mar. 21 juil. 2020 à 09:35, Enrico Olivelli <eolive...@gmail.com> a > écrit : > > > +1 (non binding) > > - verified hashes and checksums > > - built from sources and run tests (JDK14 on Linux) > > - run tests of HerdDB and some client application > > > > I only had to fix a deprecation warning, changing from > > ReduceExpressionsRule.FILTER_INSTANCE to > > CoreRules.FILTER_REDUCE_EXPRESSIONS, see [1] below > > without the change of CoreRules.FILTER_REDUCE_EXPRESSIONS all of the > tests > > of HerdDB failed with a NPE, > > I debugged the issue with a debugger and > > the ReduceExpressionsRule.FILTER_INSTANCE at runtime is null, I can't > > understand why. > > > > Not a big deal, changing to CoreRules.FILTER_REDUCE_EXPRESSIONS fixes the > > issue > > > > java.lang.NullPointerException > > at > > > > > org.apache.calcite.plan.AbstractRelOptPlanner.addRule(AbstractRelOptPlanner.java:147) > > at > > > > > org.apache.calcite.plan.volcano.VolcanoPlanner.addRule(VolcanoPlanner.java:416) > > at herddb.sql.CalcitePlanner.runPlanner(CalcitePlanner.java:576) > > at herddb.sql.CalcitePlanner.translate(CalcitePlanner.java:331) > > at herddb.core.TestUtils.scan(TestUtils.java:70) > > > > [1] > > > > > https://github.com/diennea/herddb/pull/665/files#diff-ca87d7835fc281efa58a8809669017a9R576 > > > > > > Enrico > > > > Il giorno mar 21 lug 2020 alle ore 06:12 Francis Chuang < > > francischu...@apache.org> ha scritto: > > > > > Thanks for making this release available for voting, Chunwei! > > > > > > Verified GPG Signature - OK > > > Verified SHA512 - OK > > > Ran tests per HOWTO (./gradlew check) - OK > > > Quickly skimmed release notes - Looks good, but I agree with Julian's > > > comments. > > > Spotted checked a few JARs in the Maven repository - OK > > > > > > Environment (OpenJDK:latest docker container): > > > Gradle 6.3 (via gradlew) > > > Oracle Linux Server 7.8 > > > openjdk version "14.0.2" 2020-07-14 > > > OpenJDK Runtime Environment (build 14.0.2+12-46) > > > OpenJDK 64-Bit Server VM (build 14.0.2+12-46, mixed mode, sharing) > > > > > > My vote is: +1 (binding) > > > > > > Francis > > > > > > On 21/07/2020 12:07 pm, Haisheng Yuan wrote: > > > > Environment: > > > > Mac OS X 10.15.1, JDK 1.8.0_162 > > > > > > > > - Checked signatures and checksums, OK > > > > - Ran unit tests (./gradlew build), OK > > > > > > > > +1 (binding) > > > > > > > >> * why is 4032 'breaking'? > > > > With that change, the CalcMergeRule won't match > PhysicalNode(including > > > EnumerableCalc) in VolcanoPlanner. Perhaps I should elaborate in the > > > release notes. > > > > > > > >> * why is 3786 breaking? (recomputeDigest was not present in 1.23; > the > > > >> remarks about caching digests are useful, so why aren't they in the > > > >> javadoc?) > > > > recomputeDigest() has been there since b0dab68 (2012-05-07). I will > add > > > the remarks into the javadoc after release. > > > > > > > > Thanks, > > > > Haisheng > > > > > > > > On 2020/07/21 01:14:17, Julian Hyde <jh...@apache.org> wrote: > > > >> Downloaded, checked hashes, built and ran tests on Ubuntu/JDK 14; > > > >> checked distro against git (see issue 1); reviewed release notes > (see > > > >> issue 2). > > > >> > > > >> +1 (binding) but issues 1 and 2 need to be fixed right after the > > > release. > > > >> > > > >> Issue 1. License file is not the same as in source control: > > > >> > > > >> diff -r ./LICENSE /tmp/apache-calcite-1.24.0-src/LICENSE > > > >> 177a178,189 > > > >>> > > > >>> Additional License files can be found in the 'licenses' folder > > located > > > in the same directory as the LICENSE file (i.e. this file) > > > >>> > > > >>> - Software produced outside the ASF which is available under other > > > licenses (not Apache-2.0) > > > >>> > > > >>> MIT > > > >>> * cobyism:html5shiv:3.7.2 > > > >>> * font-awesome:font-awesome-code:4.2.0 > > > >>> * gridsim:gridsim: > > > >>> * jekyll:jekyll: > > > >>> * normalize:normalize:3.0.2 > > > >>> * respond:respond:1.4.2 > > > >> > > > >> Can you fix the release instructions that the generated LICENSE > needs > > > >> to be committed (probably at the same time you revise the release > > > >> notes). > > > >> > > > >> Issue 2. Release notes > > > >> > > > >> For the 'highlights', I prefer a paragraph with hyperlinks over a > list > > > >> (see > > > > > > https://github.com/apache/calcite/blob/calcite-1.24.0-rc0/site/_docs/history.md#1180--2018-12-21 > > > ). > > > >> > > > >> Regarding categorization: > > > >> * why is 4032 'breaking'? > > > >> * why is 3786 breaking? (recomputeDigest was not present in 1.23; > the > > > >> remarks about caching digests are useful, so why aren't they in the > > > >> javadoc?) > > > >> * we need a note that a bunch of methods are deprecated in this > > > >> release and will be removed before 1.25 (see 3923, 4023 and 4079). > > > >> This will break semantic versioning in 1.25, so is a big deal. > > > >> * 4073, 3224, 4056, 4008, 3972, 4060 are listed as new features, > but I > > > >> think they are bug fixes or improved implementations > > > >> * 3946, 4089, 4087 are listed as fixes but could be listed as new > > > features > > > >> * 4075 should be under 'test suite' > > > >> * 4094 description does not need 'follow-up after review comments' > > > >> * 4086 is an upgrade, so should be in 'bug fixes', not documentation > > > >> * A few places SQL and Java keywords are not in code font (e.g. NPE, > > > >> IllegalArgumentException, RexNode, Expression, HAVING, ARRAY, MAP, > > > >> CAST) > > > >> > > > >> Julian > > > >> > > > >> On Mon, Jul 20, 2020 at 12:01 PM Michael Mior <mm...@apache.org> > > wrote: > > > >>> > > > >>> +1 > > > >>> > > > >>> Checked hash and signature and compiled and ran tests. Thanks > > Chunwei! > > > >>> > > > >>> -- > > > >>> Michael Mior > > > >>> mm...@apache.org > > > >>> > > > >>> Le lun. 20 juil. 2020 à 11:41, Chunwei Lei <chunwei.l...@gmail.com > > > > a > > > écrit : > > > >>>> > > > >>>> Hi all, > > > >>>> > > > >>>> I have created a build for Apache Calcite 1.24.0, release > > > >>>> candidate 0. > > > >>>> > > > >>>> Thanks to everyone who has contributed to this release. > > > >>>> > > > >>>> You can read the release notes here: > > > >>>> > > > > > > https://github.com/apache/calcite/blob/calcite-1.24.0-rc0/site/_docs/history.md > > > >>>> > > > >>>> The commit to be voted upon: > > > >>>> > > > > > > https://gitbox.apache.org/repos/asf?p=calcite.git;a=commit;h=4b5b9100e59ae4a43424156c9beabec6805f3d7c > > > >>>> > > > >>>> Its hash is 4b5b9100e59ae4a43424156c9beabec6805f3d7c > > > >>>> > > > >>>> Tag: > > > >>>> https://github.com/apache/calcite/tree/calcite-1.24.0-rc0 > > > >>>> > > > >>>> The artifacts to be voted on are located here: > > > >>>> > > > > https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-1.24.0-rc0 > > > >>>> (revision 40574) > > > >>>> > > > >>>> The hashes of the artifacts are as follows: > > > >>>> > > > > > > ffc7821089a444d50be228b0f0d9d8fb875c98f3b31ed0ad5a81cf5f56b9139dd353fd2c866b5bfd42a06c2a09bca579bcf6ed1e05322be1ae228fd7848f4aec > > > >>>> *apache-calcite-1.24.0-src.tar.gz > > > >>>> > > > >>>> A staged Maven repository is available for review at: > > > >>>> > > > > > > https://repository.apache.org/content/repositories/orgapachecalcite-1096/org/apache/calcite/ > > > >>>> > > > >>>> Release artifacts are signed with the following key: > > > >>>> https://www.apache.org/dist/calcite/KEYS > > > >>>> > > > >>>> N.B. > > > >>>> To create the jars and test Apache Calcite: "./gradlew build". > > > >>>> > > > >>>> If you do not have a Java environment available, you can run the > > tests > > > >>>> using docker. To do so, install docker and docker-compose, then > run > > > >>>> "docker-compose run test" from the root of the directory. > > > >>>> > > > >>>> Please vote on releasing this package as Apache Calcite 1.24.0. > > > >>>> > > > >>>> The vote is open for the next 72 hours and passes if a majority of > > at > > > >>>> least three +1 PMC votes are cast. > > > >>>> > > > >>>> [ ] +1 Release this package as Apache Calcite 1.24.0 > > > >>>> [ ] 0 I don't feel strongly about it, but I'm okay with the > release > > > >>>> [ ] -1 Do not release this package because... > > > >>>> > > > >>>> > > > >>>> Here is my vote: > > > >>>> > > > >>>> +1 (non-binding) > > > >>>> > > > >>>> > > > >>>> > > > >>>> Best, > > > >>>> Chunwei > > > >> > > > > > >
