The integration is advancing well and I am hoping to merge the PR in the coming days. To avoid unpleasant surprises, I am planning to create a new remote branch in the main calcite repo to test some things out. I will delete it as soon as I am done with the tests.
Best. Stamatis On Wed, Dec 28, 2022 at 2:47 PM Michael Mior <mm...@apache.org> wrote: > Thanks Stamatis! I haven't used SonarCloud before, but in general I think > such tools can be quite useful. > > -- > Michael Mior > mm...@apache.org > > > On Sat, Dec 24, 2022 at 4:01 PM Stamatis Zampetakis <zabe...@gmail.com> > wrote: > > > Since there were no objections, I just logged INFRA-24038 [1] and plan to > > move this forward. > > > > Let me know if you have questions or concerns regarding the adoption of > > SonarCloud. > > > > Best, > > Stamatis > > > > [1] https://issues.apache.org/jira/browse/INFRA-24038 > > > > On Sat, Dec 10, 2022 at 11:32 AM Benchao Li <libenc...@apache.org> > wrote: > > > > > Thanks Stamatis for bringing this up. > > > > > > I haven't used Sonar yet, but thanks for the demo[1] you provided, it > > looks > > > really interesting. I think it's worth a try for Calcite. > > > > > > [1] https://github.com/zabetak/calcite/pull/9 > > > > > > Alessandro Solimando <alessandro.solima...@gmail.com> 于2022年12月10日周六 > > > 02:54写道: > > > > > > > Hi all, > > > > thanks Stamatis for the proposal and the work, I am a huge fan of > Sonar > > > and > > > > I really miss it on Calcite, so a big +1 from me on this. > > > > > > > > In Hive so far we have received good feedback, so I hope it will be > > > > welcomed here too. > > > > > > > > Best regards, > > > > Alessandro > > > > > > > > On Fri, 9 Dec 2022 at 19:02, Stamatis Zampetakis <zabe...@gmail.com> > > > > wrote: > > > > > > > > > Hi all, > > > > > > > > > > I just logged CALCITE-5427 [1] for introducing code quality & > > coverage > > > > > metrics in Calcite CI. > > > > > > > > > > I added some motivation and examples under the ticket so please > have > > a > > > > look > > > > > and let me know what you think. > > > > > > > > > > If there are no objections, I will follow-up with INFRA to set > things > > > up > > > > > for the official Calcite repo. > > > > > > > > > > The integration with SonarCloud has been inspired by HIVE-26196 [2] > > > that > > > > > Alessandro put in place for Hive and has been very helpful so far. > > > > > > > > > > Best, > > > > > Stamatis > > > > > > > > > > [1] https://issues.apache.org/jira/browse/CALCITE-5427 > > > > > [2] https://issues.apache.org/jira/browse/HIVE-26196 > > > > > > > > > > > > > > > > > > -- > > > > > > Best, > > > Benchao Li > > > > > >