The DRILL-7945 blocked the release. So, I'm ready to merge the DRILL-7937 and DRILL-7940 for bugfix.
> 在 2021年6月4日,01:15,Laurent Goujon <laur...@dremio.com> 写道: > > Hey guys, > > Can we please stop changing the goal post again and again? The fact that > some of those pull requests are ready to merge should not be the sole > consideration when to do a next release candidate. > > I've been asking several times on this mailing list about what we want to > include or not, and we got an agreement several times about it, and several > times we are now having this conversation. > IMHO, I would not include DRILL-7941, DRILL-7942 and DRILL-7943: those are > new enhancements impacting Drill tests (not even the main product) and I do > not understand the rush in making them part of the release. Specifically > for the JUnit 5 update, I think the change is misleading because it looks > like it's only the introduction of JUnit5 in one test class and everything > else still uses JUnit 4, so I would hardly call it an upgrade... > > As for DRILL-7937 and DRILL-7940, the issues were open in the last 3 days > ago, but they do not seem to be regressions since 1.18.0, just gaps in what > Drill provides. Personally since we are this deep in the release, I would > also skip these one too. But if people have more contexts on those, maybe > we can agree they should be merged? > > Laurent > > >> On Thu, Jun 3, 2021 at 6:10 AM Charles Givre <cgi...@gmail.com> wrote: >> >> There are like 5 minor PRs that are approved and awaiting merge. I'd vote >> that we include them. Specifically: >> >> DRILL-7943: Update Hamcrest >> DRILL-7942: Update Mockito >> DRILL-7941: Update junit to 5.7.2 >> DRILL-7937: Parquet decimal error >> DRILL-7940: Fix Kafka Key >> >> These are all approved and can be merged. >> >> -- C >> >>>> On Jun 3, 2021, at 9:01 AM, luoc <l...@apache.org> wrote: >>> >>> >>> DRILL-7940, too >>> >>>> 在 2021年6月3日,19:57,Charles Givre <cgi...@gmail.com> 写道: >>>> >>>> -1 (Binding) >>>> >>>> I'd agree with Nick. Drill-7937 should be included in this release. >>>> -- C >>>> >>>>> On Jun 2, 2021, at 9:25 AM, Nick Stenroos-Dam <n...@project.bi> wrote: >>>>> >>>>> Vote -1 >>>>> >>>>> Can we please include DRILL-7937 >>> >> >>