Hi everyone, I'd like to merge https://github.com/apache/calcite/pull/2628 ( https://issues.apache.org/jira/browse/CALCITE-4917) which is ready if there are no objections.
Since it's just adding an extra test it's not urgent at all, so if it causes any troubles it can be postponed after the release. Best regards, Alessandro On Sat, 18 Dec 2021 at 19:19, Julian Hyde <jhyde.apa...@gmail.com> wrote: > Ah, I see Stamatis has Log4j covered: > https://issues.apache.org/jira/browse/CALCITE-4950 < > https://issues.apache.org/jira/browse/CALCITE-4950>. > > > On Dec 18, 2021, at 10:17 AM, Julian Hyde <jhyde.apa...@gmail.com> > wrote: > > > > I think we should skip CALCITE-4907. There’s still discussion in its PR, > https://github.com/apache/calcite/pull/2620 < > https://github.com/apache/calcite/pull/2620>. I promise we’ll get it in > soon after the release. > > > > In other news, there’s a new Log4j version, 2.17, addressing a new > CVE[1]. Let’s get that in. > > > > Julian > > > > [1] > https://thehackernews.com/2021/12/apache-issues-3rd-patch-to-fix-new-high.html > < > https://thehackernews.com/2021/12/apache-issues-3rd-patch-to-fix-new-high.html > > > > > >> On Dec 18, 2021, at 6:58 AM, Vladimir Ozerov <ppoze...@gmail.com > <mailto:ppoze...@gmail.com>> wrote: > >> > >> Hi Xiong, > >> > >> Thank you very much. > >> > >> Vladimir. > >> > >> Сб, 18 дек. 2021 г. в 14:32, xiong duan <nobigo...@gmail.com <mailto: > nobigo...@gmail.com>>: > >> > >>> Hi Vladimir, > >>> > >>> I have merged. > >>> > >>> Vladimir Ozerov <ppoze...@gmail.com <mailto:ppoze...@gmail.com>> > 于2021年12月18日周六 16:58写道： > >>> > >>>> Hi Xiong, > >>>> > >>>> I've just resolved the conflicts. > >>>> > >>>> Regards, > >>>> Vladimir. > >>>> > >>>> сб, 18 дек. 2021 г. в 10:48, xiong duan <nobigo...@gmail.com <mailto: > nobigo...@gmail.com>>: > >>>> > >>>>> Hi, Vladimir. > >>>>> > >>>>> The PR[https://github.com/apache/calcite/pull/2464 < > https://github.com/apache/calcite/pull/2464>] looks good. If you > >>>>> have > >>>>> free time Please resolve the conflicts. Then we can merge it. > >>>>> > >>>>> Vladimir Ozerov <ppoze...@gmail.com <mailto:ppoze...@gmail.com>> > 于2021年12月18日周六 15:20写道： > >>>>> > >>>>>> Hi, > >>>>>> > >>>>>> Could you please also merge > >>>>>> https://github.com/apache/calcite/pull/2464 < > https://github.com/apache/calcite/pull/2464> ? I guess it got stuck > >>>>>> somehow, > >>>>>> as all the comments are addressed. This issue forced us to create a > >>>> copy > >>>>> of > >>>>>> the AggregateUnionTransposeRule in our project. > >>>>>> > >>>>>> Regards, > >>>>>> Vladimir > >>>>>> > >>>>>> Сб, 18 дек. 2021 г. в 09:51, Rui Wang <amaliu...@apache.org > <mailto:amaliu...@apache.org>>: > >>>>>> > >>>>>>> Thanks Julian! > >>>>>>> > >>>>>>> Now we will just decide whether to merge CALCITE-4907 or not, then > >>> I > >>>>> will > >>>>>>> lock the main branch and start the release process. > >>>>>>> > >>>>>>> > >>>>>>> -Rui > >>>>>>> > >>>>>>> On Fri, Dec 17, 2021 at 10:29 PM Julian Hyde < > >>> jhyde.apa...@gmail.com <mailto:jhyde.apa...@gmail.com> > >>>>> > >>>>>>> wrote: > >>>>>>> > >>>>>>>> Rui, I have merged CALCITE-4946. > >>>>>>>> > >>>>>>>>> On Dec 17, 2021, at 10:21 PM, Rui Wang <amaliu...@apache.org> > >>>>> wrote: > >>>>>>>>> > >>>>>>>>> Thanks Zhe. > >>>>>>>>> > >>>>>>>>> Hi Francesco, > >>>>>>>>> > >>>>>>>>> I left a comment in your PR. Can you take a look? > >>>>>>>>> > >>>>>>>>> -Rui > >>>>>>>>> > >>>>>>>>> On Fri, Dec 17, 2021 at 3:32 PM Zhe Hu <iluff...@163.com> > >>> wrote: > >>>>>>>>> > >>>>>>>>>> Hi, Rui. > >>>>>>>>>> CALCITE-4860 will be fixed in next release, you can pass it > >>>>>> currently. > >>>>>>>>>> Thanks for your work! > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> Best, > >>>>>>>>>> ZheHu > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> On 12/18/2021 06:08，Francesco Gini<francesco.g...@gmail.com> > >>>>> wrote： > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4907 is in > >>>> review. > >>>>> I > >>>>>>>> believe > >>>>>>>>>> I have addressed all the comments. Let me know if I missed > >>>>> anything. > >>>>>>>>>> > >>>>>>>>>> On Fri, 17 Dec 2021 at 22:42, Jacques Nadeau < > >>>> jacq...@apache.org> > >>>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>> I've merged CALCITE-4948 to mitigate the concerns around > >>>>>> CALCITE-4898 > >>>>>>>> and > >>>>>>>>>> maven repositories and category x licenses. > >>>>>>>>>> > >>>>>>>>>> On Fri, Dec 17, 2021 at 1:02 PM Rui Wang < > >>> amaliu...@apache.org> > >>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>> For the list of 1.29 fix Jira, several Jira have been marked > >>> as > >>>>>>> resolved > >>>>>>>>>> since this Monday. I will do another pass today but probably > >>> not > >>>>> all > >>>>>>>>>> those > >>>>>>>>>> will make the 1.29.0 release. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> -Rui > >>>>>>>>>> > >>>>>>>>>> On Fri, Dec 17, 2021 at 11:57 AM Ruben Q L <rube...@gmail.com > >>>> > >>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>> Hello, > >>>>>>>>>> > >>>>>>>>>> FYI https://issues.apache.org/jira/browse/CALCITE-4737 has > >>> been > >>>>>>>>>> merged. > >>>>>>>>>> > >>>>>>>>>> Regards, > >>>>>>>>>> Ruben > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> On Fri, Dec 17, 2021 at 7:40 PM Julian Hyde < > >>>>> jhyde.apa...@gmail.com > >>>>>>> > >>>>>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>> Rui, > >>>>>>>>>> > >>>>>>>>>> Even though > >>> https://issues.apache.org/jira/browse/CALCITE-4898 > >>>> < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4898> is marked > >>>>>> fixed, > >>>>>>>>>> there’s some fallout regarding maven repositories and category > >>>> X. > >>>>>>>>>> That > >>>>>>>>>> needs to be sorted out before RC0. See [1]. > >>>>>>>>>> > >>>>>>>>>> I’m working on getting > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4946 > >>>>>>>>>> <https://issues.apache.org/jira/browse/CALCITE-4946> done > >>>> today. > >>>>>>>>>> Please > >>>>>>>>>> wait for that. > >>>>>>>>>> > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4885 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4885> is a big > >>>>>>>>>> refactoring > >>>>>>>>>> and, though I’d love to get it in before the release, it isn’t > >>>>> going > >>>>>>>>>> to > >>>>>>>>>> happen. Don’t wait for it. > >>>>>>>>>> > >>>>>>>>>> I also see the following as open or in progress and > >>> fix-in-1.29: > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4872 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4872> (Will > >>>> Noble) > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4908 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4908> (Woonsan > >>>> Koo) > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4704 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4704> > >>> (Stamatis) > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4737 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4737> (Thomas > >>>>> Rebele) > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4860 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4860> (Zhe Hu / > >>>>> Duan > >>>>>>>>>> Xiong) > >>>>>>>>>> * https://issues.apache.org/jira/browse/CALCITE-4907 < > >>>>>>>>>> https://issues.apache.org/jira/browse/CALCITE-4907> > >>> (Francesco > >>>>>> Gini) > >>>>>>>>>> > >>>>>>>>>> What’s the status on those? > >>>>>>>>>> > >>>>>>>>>> Julian > >>>>>>>>>> > >>>>>>>>>> [1] https://github.com/apache/calcite/pull/2622 < > >>>>>>>>>> https://github.com/apache/calcite/pull/2622> > >>>>>>>>>> > >>>>>>>>>> PS It feels churlish to bring this up, but in time-honored > >>>>> tradition > >>>>>>>>>> the > >>>>>>>>>> email subject has been ‘[DISCUSS] Towards Calcite x.y.z’. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> On Dec 14, 2021, at 10:37 AM, Rui Wang <amaliu...@apache.org> > >>>>>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>> Hi community, > >>>>>>>>>> > >>>>>>>>>> To prepare for Calcite 1.29.0 release, I will start to check > >>> the > >>>>>>>>>> JIRA > >>>>>>>>>> list > >>>>>>>>>> [1] which marks its fix version as 1.29.0 (9 JIRA as of now > >>>>> remains > >>>>>>>>>> open/in > >>>>>>>>>> progress) and also the pending PRs which looks promising to be > >>>>>>>>>> merged. > >>>>>>>>>> Please help resolve/review/merge JIRA/PRs if you want some > >>> fixes > >>>>> to > >>>>>>>>>> be > >>>>>>>>>> released. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> My current goal is to start freezing the main branch of > >>> Calcite > >>>> by > >>>>>>>>>> the > >>>>>>>>>> end > >>>>>>>>>> of this week. > >>>>>>>>>> > >>>>>>>>>> Please let me know if you have any questions. > >>>>>>>>>> > >>>>>>>>>> [1]: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>>> > >>>> > >>> > https://issues.apache.org/jira/browse/CALCITE-4941?jql=project%20%3D%20CALCITE%20AND%20status%20in%20(Open%2C%20%22In%20Progress%22)%20AND%20fixVersion%20%3D%201.29.0 > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> -Rui > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> -- > >>>>>>>>>> Francesco Gini > >>>>>>>>>> > >>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>>> > >>>> > >>> > > > >
