I think you should include 811. It adds 3 new instances of a rule but does not change the existing one.
I don't believe that Hive's test run has finished. Jesus, can you report when you have some results? Still to do are release-related changes: release notes, 741, 743. I can do the web site after the RC. I was hoping to get Maryann's patch to 793 in, but I still don't have a test case, so let's defer that one. Julian On Tue, Jul 28, 2015 at 3:23 PM, Jacques Nadeau <[email protected]> wrote: > It seems like everyone is happy with the current release. The issue > identified by Hive looks like it was a Hive fix rather than a Calcite one. > I noticed that Jesus committed one thing after my proposed release. Do > people feel like that is absolutely required in this release/the risk > involved? I'm fine including it but am initially inclined to leave it for > the next release if it isn't pressing. The jira: > https://issues.apache.org/jira/browse/CALCITE-811 > > Thoughts? > > On Mon, Jul 27, 2015 at 12:32 PM, Julian Hyde <[email protected]> wrote: > >> I’ve tried the compile against Hive[1], fixed a minor problem (related to >> SortFactory), and now Jesus is going to run the Hive suite. >> >> Jacques, If you’re writing release notes you might find my relNotes[2] >> script useful. >> >> Julian >> >> [1] https://issues.apache.org/jira/browse/HIVE-11383 >> [2] https://github.com/julianhyde/share/blob/master/tools/relNotes >> >> On Jul 24, 2015, at 7:35 PM, Jacques Nadeau <[email protected]> wrote: >> >> > I've pushed a 1.4.0-SNAPSHOT to the Apache Snapshot Repo. Framework >> > owners, please try out this branch and report back whether you see any >> > showstopping regressions. That way we can work out any kinks before we >> put >> > this to an actual release vote. >> > >> > You can to the current commit by adding the following maven snapshot >> > repository to your pom.xml: >> > >> > https://repository.apache.org/content/repositories/snapshots/ >> > >> > The version of the artifacts to reference is 1.4.0-SNAPSHOT. The >> artifacts >> > are currently based on git commit >> 5cb95e644f38da1a13d6def96eeec935b650289c. >> > >> > I'll target sending an actual release vote EOD Tuesday unless we find >> > additional issues. >> > >> > Thanks, >> > Jacques >> >>
