Yes, this looks like a regression. I can try and help fix it. On Thu, Jun 3, 2021 at 12:15 PM <volody...@apache.org> wrote:
> Hi all, > > Thanks, Laurent for creating this release candidate! > > I have started verifying the release, but for now my vote is -1 (binding) > because of the following issue: DRILL-7945 < > https://issues.apache.org/jira/browse/DRILL-7945> > > It is definitely a regression and it might break some important Drill > functionality so I'm considering it as a blocker for the release. > > Kind regards, > Volodymyr Vysotskyi > > > On 2021/06/03 17:14:37, Laurent Goujon <l...@dremio.com> wrote: > > 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 <cg...@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 <lu...@apache.org> wrote:> > > > >> > > > >> > > > > DRILL-7940, too> > > > >> > > > >> 在 2021年6月3日,19:57,Charles Givre <cg...@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 <ni...@project.bi> > wrote:> > > > >>>> > > > >>> Vote -1> > > > >>>> > > > >>> Can we please include DRILL-7937> > > > >> > > >> > > >> > >