Return-Path: X-Original-To: apmail-drill-dev-archive@www.apache.org Delivered-To: apmail-drill-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7030F18B32 for ; Fri, 11 Mar 2016 00:10:01 +0000 (UTC) Received: (qmail 39787 invoked by uid 500); 11 Mar 2016 00:10:01 -0000 Delivered-To: apmail-drill-dev-archive@drill.apache.org Received: (qmail 39739 invoked by uid 500); 11 Mar 2016 00:10:01 -0000 Mailing-List: contact dev-help@drill.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@drill.apache.org Delivered-To: mailing list dev@drill.apache.org Received: (qmail 39728 invoked by uid 99); 11 Mar 2016 00:10:01 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 11 Mar 2016 00:10:01 +0000 Received: from mail-pf0-f175.google.com (mail-pf0-f175.google.com [209.85.192.175]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 9E00A1A010E for ; Fri, 11 Mar 2016 00:10:00 +0000 (UTC) Received: by mail-pf0-f175.google.com with SMTP id n5so42447155pfn.2 for ; Thu, 10 Mar 2016 16:10:00 -0800 (PST) X-Gm-Message-State: AD7BkJI2U8zupygakQGla6jzHTa5Zf4g6PV6CpFW9FaL0PVYKdJU/DAtjDPSajns70H0n6ypr5rCSmRdMtH1dpUU MIME-Version: 1.0 X-Received: by 10.98.40.200 with SMTP id o191mr9159796pfo.83.1457654999895; Thu, 10 Mar 2016 16:09:59 -0800 (PST) Received: by 10.66.9.102 with HTTP; Thu, 10 Mar 2016 16:09:59 -0800 (PST) In-Reply-To: References: <25CAD7FC-77D9-4563-AF58-49EEC4FEED40@maprtech.com> <84392C72-19D3-43E1-866F-E4DA7D57690B@maprtech.com> Date: Thu, 10 Mar 2016 16:09:59 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: Time for the 1.6 Release From: Parth Chandra To: dev Content-Type: multipart/alternative; boundary=001a113549d47971cc052dbac0fc --001a113549d47971cc052dbac0fc Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi guys, I've created the 1.6.0 branch and will be rolling out the first release candidate as soon as I get the go ahead from the QA team, in the next couple of hours. Thanks Parth On Thu, Mar 10, 2016 at 3:20 PM, Parth Chandra wrote: > It is usually not a good idea to try to rush in a patch at the last > minute. One of the reasons for having monthly releases is so people don't > have to wait too long for fixes and developers don't rush fixes in. > QA is almost done with their validation, so I'm afraid this might have to > go into the next release. > > On Thu, Mar 10, 2016 at 1:19 PM, Jason Altekruse wrote= : > >> I hadn't actually tested out the patch, what I had said was that I could >> add a flag to make avro files behave like parquet and JSON, without sche= ma >> validation. The patch made it so the behavior of directories would be >> different from that of individual files, removing the schema validation.= I >> tried applying it just now and it still doesn't appear to make the dirN >> columns work, but I don't understand why. I will try to take a look >> tonight >> and post a patch. It will be up to Parth if he wants to put it in the >> release once the full fix is merged. >> >> Jason Altekruse >> Software Engineer at Dremio >> Apache Drill Committer >> >> On Thu, Mar 10, 2016 at 1:09 AM, Stef=C3=A1n Baxter > > >> wrote: >> >> > Hi, >> > >> > This issue is still unresolved: >> > https://issues.apache.org/jira/browse/DRILL-4120 >> > >> > It would mean a great deal to us if it was. >> > The solution is, as I understood Jason and Jacques, ready and only >> needs to >> > be merged. >> > >> > Regards, >> > -Stef=C3=A1n >> > >> > On Thu, Mar 10, 2016 at 3:50 AM, Parth Chandra >> wrote: >> > >> > > Hi everyone, >> > > >> > > Just a note to update everyone that the QA team is testing out th= e >> > build >> > > from master. >> > > There are no further commits expected for the 1.6.0 release. >> > > The repo is open for commits but try not to break anything :) >> > > >> > > >> > > Parth >> > > >> > > On Tue, Mar 8, 2016 at 5:16 PM, Parth Chandra >> wrote: >> > > >> > > > Okay we are down to the final one - >> > > > >> > > > DRILL-4482 - Avro no longer selects data correctly from a >> > > > sub-structure.(Jason) >> > > > >> > > > Note that MapR QA team is going to start testing 1.6 snapshot now >> > before >> > > I >> > > > roll out the release candidate. DRILL-4482 can be merged in later >> as it >> > > is >> > > > not likely to affect the Hopefully there will be no show stoppers >> (. >> > > > >> > > > The plan is to roll out the release candidate by Thursday. >> > > > >> > > > Thanks >> > > > >> > > > Parth >> > > > >> > > > >> > > > On Tue, Mar 8, 2016 at 9:31 AM, Parth Chandra >> > wrote: >> > > > >> > > >> OK, let's leave it out then. >> > > >> >> > > >> On Tue, Mar 8, 2016 at 9:25 AM, Jason Altekruse < >> > > altekrusejason@gmail.com >> > > >> > wrote: >> > > >> >> > > >>> To be honest I was expecting a longer review cycle so I hadn't r= un >> > the >> > > >>> unit >> > > >>> tests before posting it for review. There were only very minor >> > > functional >> > > >>> changes, so I wasn't thinking it would be an issue, and I was >> > > >>> anticipating >> > > >>> having to update the patch before merging it. I could update the >> test >> > > >>> that >> > > >>> is failing but I don't see much sense in trying to get it into t= he >> > > >>> release >> > > >>> because it only introduces new tests and some small core >> refactoring. >> > > >>> >> > > >>> I'm all for getting it merged so everyone can start using it, I >> just >> > > >>> think >> > > >>> it doesn't really matter if it happens on the release branch or >> back >> > on >> > > >>> master once we cut a release branch. >> > > >>> >> > > >>> I would rather try to focus on getting the Avro issues resolved, >> > which >> > > is >> > > >>> what I'm working on right now. >> > > >>> >> > > >>> - Jason >> > > >>> >> > > >>> On Tue, Mar 8, 2016 at 8:58 AM, Parth Chandra >> > > wrote: >> > > >>> >> > > >>> > Sounds good Jason. Let's finalize this in the hangout. >> > > >>> > Do you have the expected plans for the failing tests? If so ca= n >> you >> > > >>> update >> > > >>> > those and put in a pull request and we'll merge and run the >> tests. >> > > >>> > Any reason for the operator test framework to be punted? You >> have a >> > > +1 >> > > >>> to >> > > >>> > merge it. >> > > >>> > >> > > >>> > >> > > >>> > >> > > >>> > On Mon, Mar 7, 2016 at 9:33 PM, Khurram Faraaz < >> > kfaraaz@maprtech.com >> > > > >> > > >>> > wrote: >> > > >>> > >> > > >>> > > We should update the expected results (i.e. the expected que= ry >> > plan >> > > >>> in >> > > >>> > this >> > > >>> > > case) and not mark them as Failing. We do not have a Failing >> test >> > > >>> > directory >> > > >>> > > today. >> > > >>> > > >> > > >>> > > If we are sure the expected query plans for these tests are >> > > incorrect >> > > >>> > > (today), we should go ahead and update the expected results >> files >> > > >>> with >> > > >>> > the >> > > >>> > > correct query plans. >> > > >>> > > >> > > >>> > > Functional/ctas/ctas_auto_partition/existing_partition_ >> > > >>> > > pruning/hierarchical/plan/count2_MD-185.q >> > > >>> > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_2= .q >> > > >>> > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_4= .q >> > > >>> > > >> > > >>> > > Thanks, >> > > >>> > > Khurram >> > > >>> > > >> > > >>> > > On Tue, Mar 8, 2016 at 8:58 AM, Jason Altekruse < >> > > >>> > altekrusejason@gmail.com> >> > > >>> > > wrote: >> > > >>> > > >> > > >>> > > > The cutoff time sounds good to me. I don't think I'm too >> > > >>> overburdened, >> > > >>> > I >> > > >>> > > > mostly tried putting together a merge branch today and had >> to >> > > kick >> > > >>> a >> > > >>> > few >> > > >>> > > > things out as I found issues. >> > > >>> > > > >> > > >>> > > > As we are not running the full regression cluster at >> Dremio, it >> > > >>> would >> > > >>> > be >> > > >>> > > > helpful if someone could merge Jacques patch (DRILL-4474) >> after >> > > >>> running >> > > >>> > > the >> > > >>> > > > complete set of tests. Could someone at MapR pick up the >> test >> > > >>> updates >> > > >>> > and >> > > >>> > > > merging this patch? >> > > >>> > > > >> > > >>> > > > These are the failures I was seeing so far, my fix was jus= t >> > going >> > > >>> to be >> > > >>> > > to >> > > >>> > > > mark the tests as failing: >> > > >>> > > > >> > > >>> > > > >> > > >>> > > >> > > >>> > >> > > >>> >> > > >> > >> Functional/ctas/ctas_auto_partition/existing_partition_pruning/hierarchi= cal/plan/count2_MD-185.q >> > > >>> > > > >> Functional/ctas/ctas_auto_partition/general/plan/drill3947_2.q >> > > >>> > > > >> Functional/ctas/ctas_auto_partition/general/plan/drill3947_4.q >> > > >>> > > > >> > > >>> > > > I will just go ahead and merge the other two fixes as I ha= d >> a >> > > clean >> > > >>> > unit >> > > >>> > > > test run and neither of the patches will have any impact o= n >> the >> > > >>> > > regression >> > > >>> > > > tests. One is a build fix, that only changes behavior when >> > > running >> > > >>> the >> > > >>> > > > release profile and the other is a unit test fix. >> > > >>> > > > >> > > >>> > > > I had punted the operator unit tests out of the release, s= o >> > those >> > > >>> can >> > > >>> > be >> > > >>> > > > removed from your list. >> > > >>> > > > >> > > >>> > > > I will start looking back at both of the Avro issues right >> > after >> > > >>> > merging. >> > > >>> > > > >> > > >>> > > > On Mon, Mar 7, 2016 at 6:32 PM, Parth Chandra < >> > > >>> pchandra@maprtech.com> >> > > >>> > > > wrote: >> > > >>> > > > >> > > >>> > > > > The train is about to leave :). I'd like to propose a >> cutoff >> > > for >> > > >>> > > > tomorrow 5 >> > > >>> > > > > pm. >> > > >>> > > > > (This will be a full week after the initial mail so >> plenty of >> > > >>> time >> > > >>> > for >> > > >>> > > > > everyone to get their issues aired.) >> > > >>> > > > > >> > > >>> > > > > Here are the open items (that I am aware of) - >> > > >>> > > > > >> > > >>> > > > > Waiting to be merged >> > > >>> > > > > DRILL-4483/pr 411 (Fix text plan regression in query >> > profiles) >> > > >>> > (Venki) >> > > >>> > > > > DRILL-4437 (and others)/pr 394 (Operator unit test >> > framework). >> > > >>> > (Jason) >> > > >>> > > > > DRILL-4332/pr 389 (Make vector comparison order stable i= n >> > test >> > > >>> > > framework) >> > > >>> > > > > (?) >> > > >>> > > > > DRILL-4474/pr 406 (Ensure that ConvertCountToDirectScan >> only >> > > >>> pushes >> > > >>> > > > through >> > > >>> > > > > project when project is trivial.) - Tests need to be fix= ed >> > > >>> (Jason) >> > > >>> > > > > >> > > >>> > > > > Need review - >> > > >>> > > > > DRILL-4375/pr 402 (Fix the maven release profile) - >> (Jason - >> > > >>> patch >> > > >>> > > > > reviewed. Merge) >> > > >>> > > > > DRILL-4486/pr 412 (Fix expression serialization escaping= ) >> > (Need >> > > >>> are >> > > >>> > > > > reviewer) >> > > >>> > > > > DRILL-4485 - Update MapR profile to use MapR 5.1.0 >> > > (Parth/Aditya) >> > > >>> > > > > >> > > >>> > > > > Regression >> > > >>> > > > > DRILL-4482 - Avro no longer selects data correctly from = a >> > > >>> > > sub-structure. >> > > >>> > > > No >> > > >>> > > > > patch available. (Jason) >> > > >>> > > > > >> > > >>> > > > > Dropped >> > > >>> > > > > DRILL-4372/pr 377(?) (Drill Operators and Functions shou= ld >> > > >>> correctly >> > > >>> > > > expose >> > > >>> > > > > their types within Calcite.) Pushed out because of test >> > > failures >> > > >>> > > > > >> > > >>> > > > > Jason, at the moment it looks like you've volunteered fo= r >> too >> > > >>> much. >> > > >>> > Can >> > > >>> > > > we >> > > >>> > > > > get other folks to help out? >> > > >>> > > > > >> > > >>> > > > > Thanks >> > > >>> > > > > >> > > >>> > > > > Parth >> > > >>> > > > > >> > > >>> > > > > >> > > >>> > > > > On Mon, Mar 7, 2016 at 5:32 PM, Jason Altekruse < >> > > >>> > > > altekrusejason@gmail.com> >> > > >>> > > > > wrote: >> > > >>> > > > > >> > > >>> > > > > > 4474 is actually revealing a few invalid tests in the >> > > >>> Regression >> > > >>> > > suite >> > > >>> > > > > that >> > > >>> > > > > > test for the current incorrect plans. The fix should b= e >> > > >>> included in >> > > >>> > > the >> > > >>> > > > > > release, but I will post a PR on the regression suite = to >> > > >>> update the >> > > >>> > > > tests >> > > >>> > > > > > before I push it. >> > > >>> > > > > > >> > > >>> > > > > > On Mon, Mar 7, 2016 at 4:44 PM, Steven Phillips < >> > > >>> steven@dremio.com >> > > >>> > > >> > > >>> > > > > wrote: >> > > >>> > > > > > >> > > >>> > > > > > > DRILL-4486 is a pretty simple fix. Without it, >> currently >> > > some >> > > >>> > regex >> > > >>> > > > > > queries >> > > >>> > > > > > > will fail. >> > > >>> > > > > > > >> > > >>> > > > > > > I think we should include it in the release. >> > > >>> > > > > > > >> > > >>> > > > > > > >> > > >>> > > > > > > https://github.com/apache/drill/pull/412 >> > > >>> > > > > > > >> > > >>> > > > > > > On Mon, Mar 7, 2016 at 2:15 PM, Jason Altekruse < >> > > >>> > > > > > altekrusejason@gmail.com> >> > > >>> > > > > > > wrote: >> > > >>> > > > > > > >> > > >>> > > > > > > > There is a small test issue with some of the >> > refactoring >> > > >>> that >> > > >>> > > > > > accompanied >> > > >>> > > > > > > > the operator unit tests. These don't change any >> > > user-facing >> > > >>> > > > behavior, >> > > >>> > > > > > so >> > > >>> > > > > > > I >> > > >>> > > > > > > > don't think it really needs to get into the >> release. I >> > > >>> will be >> > > >>> > > > > working >> > > >>> > > > > > to >> > > >>> > > > > > > > merge them into master after we cut the release >> branch. >> > > >>> > > > > > > > >> > > >>> > > > > > > > The change to update the avatica JDBC driver versi= on >> > also >> > > >>> does >> > > >>> > > not >> > > >>> > > > > make >> > > >>> > > > > > > any >> > > >>> > > > > > > > behavior changes, so I think it also makes sense t= o >> > keep >> > > >>> it off >> > > >>> > > the >> > > >>> > > > > > > release >> > > >>> > > > > > > > branch. >> > > >>> > > > > > > > >> > > >>> > > > > > > > I will be merging the fix for 4375 the maven relea= se >> > > >>> profile, >> > > >>> > > 4474 >> > > >>> > > > > > wrong >> > > >>> > > > > > > > results with incorrect creation of DirectScan and >> 4332 >> > > >>> fixing a >> > > >>> > > > unit >> > > >>> > > > > > test >> > > >>> > > > > > > > to work in JDK 8, after another test run. >> > > >>> > > > > > > > >> > > >>> > > > > > > > On Mon, Mar 7, 2016 at 1:53 PM, Venki Korukanti < >> > > >>> > > > > > > venki.korukanti@gmail.com >> > > >>> > > > > > > > > >> > > >>> > > > > > > > wrote: >> > > >>> > > > > > > > >> > > >>> > > > > > > > > WebUI profile issue: this is a regression cause = by >> > > >>> > refactoring >> > > >>> > > of >> > > >>> > > > > > > Calcite >> > > >>> > > > > > > > > integration code (DRILL-4465) which sets the tex= t >> > plan >> > > >>> only >> > > >>> > if >> > > >>> > > > > debug >> > > >>> > > > > > is >> > > >>> > > > > > > > > enabled. Will submit a patch soon. >> > > >>> > > > > > > > > >> > > >>> > > > > > > > > On Mon, Mar 7, 2016 at 1:29 PM, Sudheesh Katkam = < >> > > >>> > > > > > skatkam@maprtech.com> >> > > >>> > > > > > > > > wrote: >> > > >>> > > > > > > > > >> > > >>> > > > > > > > > > Thanks for clarifying Jacques. >> > > >>> > > > > > > > > > >> > > >>> > > > > > > > > > I haven=E2=80=99t looked into the fix for DRIL= L-4384; I >> > > >>> reopened it >> > > >>> > > > > because >> > > >>> > > > > > > the >> > > >>> > > > > > > > > > description mentioned =E2=80=9Cvisualized plan= =E2=80=9D section >> is >> > > >>> (also) >> > > >>> > > > empty. >> > > >>> > > > > > > > > > >> > > >>> > > > > > > > > > Thank you, >> > > >>> > > > > > > > > > Sudheesh >> > > >>> > > > > > > > > > >> > > >>> > > > > > > > > > > On Mar 7, 2016, at 1:08 PM, Jacques Nadeau < >> > > >>> > > > jacques@dremio.com >> > > >>> > > > > > >> > > >>> > > > > > > > wrote: >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > > The new bug (currently filed under DRILL-438= 4) >> > is a >> > > >>> > > > completely >> > > >>> > > > > > > > > different >> > > >>> > > > > > > > > > > bug than the original (original one has to d= o >> > with >> > > >>> > profile >> > > >>> > > > > > metrics, >> > > >>> > > > > > > > > this >> > > >>> > > > > > > > > > > has to do with plan text). I try to look at = it >> > > >>> tonight if >> > > >>> > > > noone >> > > >>> > > > > > can >> > > >>> > > > > > > > get >> > > >>> > > > > > > > > > to >> > > >>> > > > > > > > > > > it sooner. >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > > -- >> > > >>> > > > > > > > > > > Jacques Nadeau >> > > >>> > > > > > > > > > > CTO and Co-Founder, Dremio >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > > On Mon, Mar 7, 2016 at 12:37 PM, Parth >> Chandra < >> > > >>> > > > > > > > pchandra@maprtech.com> >> > > >>> > > > > > > > > > > wrote: >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > >> DRILL-4384 is a blocker for the release >> though >> > > >>> > > > > > > > > > >> >> > > >>> > > > > > > > > > >> On Mon, Mar 7, 2016 at 12:01 PM, Sudheesh >> > Katkam < >> > > >>> > > > > > > > > skatkam@maprtech.com> >> > > >>> > > > > > > > > > >> wrote: >> > > >>> > > > > > > > > > >> >> > > >>> > > > > > > > > > >>> I reopened DRILL-4384 < >> > > >>> > > > > > > > > > >> https://issues.apache.org/jira/browse/DRILL-4384> >> > > >>> > > > > > > > > > >>> (blocker); it is assigned to Jacques. >> > > >>> > > > > > > > > > >>> >> > > >>> > > > > > > > > > >>> On the latest master, the visualized and >> > physical >> > > >>> plan >> > > >>> > > tabs >> > > >>> > > > > on >> > > >>> > > > > > > web >> > > >>> > > > > > > > UI >> > > >>> > > > > > > > > > are >> > > >>> > > > > > > > > > >>> empty. >> > > >>> > > > > > > > > > >>> >> > > >>> > > > > > > > > > >>> Thank you, >> > > >>> > > > > > > > > > >>> Sudheesh >> > > >>> > > > > > > > > > >>> >> > > >>> > > > > > > > > > >>>> On Mar 7, 2016, at 11:39 AM, Jason >> Altekruse < >> > > >>> > > > > > > > > > altekrusejason@gmail.com >> > > >>> > > > > > > > > > >>> >> > > >>> > > > > > > > > > >>> wrote: >> > > >>> > > > > > > > > > >>>> >> > > >>> > > > > > > > > > >>>> I don't know if there are any specific ti= me >> > > >>> > constraints >> > > >>> > > > for >> > > >>> > > > > > > > getting >> > > >>> > > > > > > > > > out >> > > >>> > > > > > > > > > >>> the >> > > >>> > > > > > > > > > >>>> release, but I'm inclined to go with Vick= y >> on >> > > >>> > > DRILL-4477, >> > > >>> > > > at >> > > >>> > > > > > > least >> > > >>> > > > > > > > > > some >> > > >>> > > > > > > > > > >>>> investigation into the scope of a fix >> would be >> > > >>> good. I >> > > >>> > > > think >> > > >>> > > > > > > it's >> > > >>> > > > > > > > > > >>>> reasonably big problem whether it's a >> > regression >> > > >>> or >> > > >>> > not. >> > > >>> > > > > > > > > > >>>> >> > > >>> > > > > > > > > > >>>> On Mon, Mar 7, 2016 at 11:35 AM, Zelaine >> Fong >> > < >> > > >>> > > > > > > zfong@maprtech.com >> > > >>> > > > > > > > > >> > > >>> > > > > > > > > > >>> wrote: >> > > >>> > > > > > > > > > >>>> >> > > >>> > > > > > > > > > >>>>> Hakim, >> > > >>> > > > > > > > > > >>>>> >> > > >>> > > > > > > > > > >>>>> Yes, we'll include this in the release. >> > > >>> > > > > > > > > > >>>>> >> > > >>> > > > > > > > > > >>>>> -- Zelaine >> > > >>> > > > > > > > > > >>>>> >> > > >>> > > > > > > > > > >>>>> On Mon, Mar 7, 2016 at 9:31 AM, Abdel >> Hakim >> > > >>> Deneche < >> > > >>> > > > > > > > > > >>> adeneche@maprtech.com >> > > >>> > > > > > > > > > >>>>>> >> > > >>> > > > > > > > > > >>>>> wrote: >> > > >>> > > > > > > > > > >>>>> >> > > >>> > > > > > > > > > >>>>>> If we still have time, I would like to >> > include >> > > >>> > > > DRILL-4457 >> > > >>> > > > > > [1], >> > > >>> > > > > > > > > it's >> > > >>> > > > > > > > > > a >> > > >>> > > > > > > > > > >>>>> wrong >> > > >>> > > > > > > > > > >>>>>> results issue, I already have a fix and >> it's >> > > >>> passing >> > > >>> > > all >> > > >>> > > > > > > tests, >> > > >>> > > > > > > > I >> > > >>> > > > > > > > > am >> > > >>> > > > > > > > > > >>> just >> > > >>> > > > > > > > > > >>>>>> waiting for a review [2] >> > > >>> > > > > > > > > > >>>>>> >> > > >>> > > > > > > > > > >>>>>> >> > > >>> > > > > > > > > > >>>>>> [1] >> > > >>> > https://issues.apache.org/jira/browse/DRILL-4457 >> > > >>> > > > > > > > > > >>>>>> [2] >> > https://github.com/apache/drill/pull/410 >> > > >>> > > > > > > > > > >>>>>> >> > > >>> > > > > > > > > > >>>>>> On Mon, Mar 7, 2016 at 4:50 PM, Parth >> > Chandra >> > > < >> > > >>> > > > > > > > parthc@apache.org> >> > > >>> > > > > > > > > > >>> wrote: >> > > >>> > > > > > > > > > >>>>>> >> > > >>> > > > > > > > > > >>>>>>> Hi guys, >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> I'm still waiting for the following to >> be >> > > >>> > > > reviewed/merged >> > > >>> > > > > > by >> > > >>> > > > > > > > > today. >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> DRILL-4437 (and others)/pr 394 (Operat= or >> > unit >> > > >>> test >> > > >>> > > > > > > framework). >> > > >>> > > > > > > > > > >> Waiting >> > > >>> > > > > > > > > > >>>>> to >> > > >>> > > > > > > > > > >>>>>>> be merged (Jason) >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> DRILL-4372/pr 377(?) (Drill Operators >> and >> > > >>> Functions >> > > >>> > > > > should >> > > >>> > > > > > > > > > correctly >> > > >>> > > > > > > > > > >>>>>> expose >> > > >>> > > > > > > > > > >>>>>>> their types within Calcite.) - (Jinfen= g >> to >> > > >>> review) >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> DRILL-4313/pr 396 (Improved client >> > > >>> randomization. >> > > >>> > > > Update >> > > >>> > > > > > > JIRA >> > > >>> > > > > > > > > with >> > > >>> > > > > > > > > > >>>>>>> warnings about using the feature ) >> > > >>> > > > (Hanifi/Sudheesh/Paul >> > > >>> > > > > - >> > > >>> > > > > > > > patch >> > > >>> > > > > > > > > > >>>>>> reviewed. >> > > >>> > > > > > > > > > >>>>>>> No +1) >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> DRILL-4375/pr 402 (Fix the maven relea= se >> > > >>> profile) - >> > > >>> > > > > (Jason >> > > >>> > > > > > - >> > > >>> > > > > > > > > patch >> > > >>> > > > > > > > > > >>>>>>> reviewed. Ready to merge?) >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> Thanks >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> Parth >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>> On Sun, Mar 6, 2016 at 12:01 PM, Adity= a >> < >> > > >>> > > > > > > > adityakishore@gmail.com >> > > >>> > > > > > > > > > >> > > >>> > > > > > > > > > >>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>>> DRILL-4375/pr 402 - reviewed. >> > > >>> > > > > > > > > > >>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>> On Sun, Mar 6, 2016 at 12:48 AM, Stef= =C3=A1n >> > > >>> Baxter < >> > > >>> > > > > > > > > > >>>>>>> stefan@activitystream.com> >> > > >>> > > > > > > > > > >>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> Please review this and then consider >> as a >> > > >>> > potential >> > > >>> > > > > > > blocker: >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> https://issues.apache.org/jira/browse/DRILL-4482 >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> On Sat, Mar 5, 2016 at 3:15 AM, Part= h >> > > >>> Chandra < >> > > >>> > > > > > > > > parthc@apache.org >> > > >>> > > > > > > > > > > >> > > >>> > > > > > > > > > >>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> Okay here's the list of JIRA's >> > pending. >> > > It >> > > >>> > looks >> > > >>> > > > > like >> > > >>> > > > > > we >> > > >>> > > > > > > > > need >> > > >>> > > > > > > > > > >>>>> to >> > > >>> > > > > > > > > > >>>>>>> get >> > > >>> > > > > > > > > > >>>>>>>>> some >> > > >>> > > > > > > > > > >>>>>>>>>> more time to get the PRs still unde= r >> > > review >> > > >>> > > merged, >> > > >>> > > > so >> > > >>> > > > > > > I'll >> > > >>> > > > > > > > > wait >> > > >>> > > > > > > > > > >>>>>> over >> > > >>> > > > > > > > > > >>>>>>>> the >> > > >>> > > > > > > > > > >>>>>>>>>> weekend. >> > > >>> > > > > > > > > > >>>>>>>>>> It looks like the PRs that no >> reviewers >> > > >>> assigned >> > > >>> > > in >> > > >>> > > > > the >> > > >>> > > > > > > list >> > > >>> > > > > > > > > > >>>>> below >> > > >>> > > > > > > > > > >>>>>>> may >> > > >>> > > > > > > > > > >>>>>>>>> not >> > > >>> > > > > > > > > > >>>>>>>>>> make it into the release. >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> Thanks >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> Committed for 1.6 - >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4384 - Query profile is missi= ng >> > > >>> important >> > > >>> > > > > > > information >> > > >>> > > > > > > > on >> > > >>> > > > > > > > > > >>>>>> WebUi >> > > >>> > > > > > > > > > >>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 support= ) >> - >> > > >>> Merged. >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4410/pr 380 (listvector shoul= d >> > > >>> initiatlize >> > > >>> > > > > > bits...) >> > > >>> > > > > > > - >> > > >>> > > > > > > > > > >>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4383/pr 375 (Allow custom >> configs >> > > for >> > > >>> S3, >> > > >>> > > > > > Kerberos, >> > > >>> > > > > > > > > etc) - >> > > >>> > > > > > > > > > >>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4465/pr 401 (Simplify Calcite >> > > parsing >> > > >>> & >> > > >>> > > > planning >> > > >>> > > > > > > > > > >>>>>> integration) - >> > > >>> > > > > > > > > > >>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4467 - Baselines need updatin= g. >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4449/pr 389 (Wrong results wh= en >> > > >>> metadata >> > > >>> > > cache >> > > >>> > > > > is >> > > >>> > > > > > > > > used..) >> > > >>> > > > > > > > > > >>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4437 (and others)/pr 394 >> (Operator >> > > >>> unit >> > > >>> > test >> > > >>> > > > > > > > framework). >> > > >>> > > > > > > > > > >>>>>>> Waiting >> > > >>> > > > > > > > > > >>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>> be merged >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4449/pr 389 (Wrong results wh= en >> > > >>> metadata >> > > >>> > > cache >> > > >>> > > > > is >> > > >>> > > > > > > > > used..) >> > > >>> > > > > > > > > > - >> > > >>> > > > > > > > > > >>>>>>>>> Waiting >> > > >>> > > > > > > > > > >>>>>>>>>> to be merged >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> Need review - >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4281/pr 400 (Drill should >> support >> > > >>> inbound >> > > >>> > > > > > > > impersonation) >> > > >>> > > > > > > > > > >>>>>>> (Jacques >> > > >>> > > > > > > > > > >>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4372/pr 377(?) (Drill Operato= rs >> > and >> > > >>> > > Functions >> > > >>> > > > > > should >> > > >>> > > > > > > > > > >>>>>> correctly >> > > >>> > > > > > > > > > >>>>>>>>> expose >> > > >>> > > > > > > > > > >>>>>>>>>> their types within Calcite.) - >> (Jinfeng >> > to >> > > >>> > review) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4313/pr 396 (Improved client >> > > >>> > randomization. >> > > >>> > > > > > Update >> > > >>> > > > > > > > JIRA >> > > >>> > > > > > > > > > >>>>> with >> > > >>> > > > > > > > > > >>>>>>>>>> warnings about using the feature ) >> > > >>> > > > > (Hanifi/Sudheesh/Paul >> > > >>> > > > > > > to >> > > >>> > > > > > > > > > >>>>>> review.) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4375/pr 402 (Fix the maven >> release >> > > >>> > profile) >> > > >>> > > - >> > > >>> > > > > > > (Aditya >> > > >>> > > > > > > > to >> > > >>> > > > > > > > > > >>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4452/pr 395 (Update Avatica >> Driver >> > > to >> > > >>> > latest >> > > >>> > > > > > > Calcite) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4332/pr 389 (Make vector >> > comparison >> > > >>> order >> > > >>> > > > stable >> > > >>> > > > > > in >> > > >>> > > > > > > > test >> > > >>> > > > > > > > > > >>>>>>>> framework) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4411/pr 381 (hash join >> over-memory >> > > >>> > > condition) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4387/pr 379 (GroupScan should >> not >> > > use >> > > >>> star >> > > >>> > > > > column) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4184/pr 372 (support variable >> > length >> > > >>> > decimal >> > > >>> > > > > > fields >> > > >>> > > > > > > in >> > > >>> > > > > > > > > > >>>>>> parquet) >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4120 - dir0 does not work whe= n >> the >> > > >>> > directory >> > > >>> > > > > > > structure >> > > >>> > > > > > > > > > >>>>>> contains >> > > >>> > > > > > > > > > >>>>>>>>> Avro >> > > >>> > > > > > > > > > >>>>>>>>>> files - Partial patch available. >> > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4203/pr 341 (fix dates writte= n >> > into >> > > >>> > parquet >> > > >>> > > > > files >> > > >>> > > > > > to >> > > >>> > > > > > > > > > >>>>> conform >> > > >>> > > > > > > > > > >>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>> parquet format spec) >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>> On Fri, Mar 4, 2016 at 6:24 PM, >> Jacques >> > > >>> Nadeau < >> > > >>> > > > > > > > > > >>>>> jacques@dremio.com >> > > >>> > > > > > > > > > >>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>> Awesome. Thanks Chun! >> > > >>> > > > > > > > > > >>>>>>>>>>> On Mar 4, 2016 5:51 PM, "Chun >> Chang" < >> > > >>> > > > > > > cchang@maprtech.com> >> > > >>> > > > > > > > > > >>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>> Jacques submitted a PR for fixing >> the >> > > >>> failed >> > > >>> > > > > > baselines. >> > > >>> > > > > > > > I've >> > > >>> > > > > > > > > > >>>>>>> merged >> > > >>> > > > > > > > > > >>>>>>>>>> them >> > > >>> > > > > > > > > > >>>>>>>>>>>> into automation master and >> confirmed >> > the >> > > >>> > failed >> > > >>> > > > > tests >> > > >>> > > > > > > are >> > > >>> > > > > > > > > all >> > > >>> > > > > > > > > > >>>>>>>> passing >> > > >>> > > > > > > > > > >>>>>>>>>>> now. >> > > >>> > > > > > > > > > >>>>>>>>>>>> Thanks. >> > > >>> > > > > > > > > > >>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>> -Chun >> > > >>> > > > > > > > > > >>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>> On Thu, Mar 3, 2016 at 10:48 PM, >> > Jacques >> > > >>> > Nadeau >> > > >>> > > < >> > > >>> > > > > > > > > > >>>>>>>> jacques@dremio.com> >> > > >>> > > > > > > > > > >>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>> I think we need to include >> DRILL-4467 >> > > >>> > > > > > > > > > >>>>>>>>>>>>> < >> > > >>> > > > https://issues.apache.org/jira/browse/DRILL-4467 >> > > >>> > > > > >. >> > > >>> > > > > > I >> > > >>> > > > > > > > > > >>>>> think >> > > >>> > > > > > > > > > >>>>>> it >> > > >>> > > > > > > > > > >>>>>>>> is >> > > >>> > > > > > > > > > >>>>>>>>> a >> > > >>> > > > > > > > > > >>>>>>>>>>> one >> > > >>> > > > > > > > > > >>>>>>>>>>>>> line patch and it provides >> > > unpredictable >> > > >>> > plans >> > > >>> > > > at a >> > > >>> > > > > > > > minimum >> > > >>> > > > > > > > > > >>>>>> but >> > > >>> > > > > > > > > > >>>>>>>> may >> > > >>> > > > > > > > > > >>>>>>>>>>> also >> > > >>> > > > > > > > > > >>>>>>>>>>>>> present invalid result. Still >> need to >> > > >>> think >> > > >>> > > > through >> > > >>> > > > > > the >> > > >>> > > > > > > > > > >>>>>> second >> > > >>> > > > > > > > > > >>>>>>>>> half. >> > > >>> > > > > > > > > > >>>>>>>>>>> I've >> > > >>> > > > > > > > > > >>>>>>>>>>>>> seen this plan instability in >> some of >> > > my >> > > >>> > recent >> > > >>> > > > > test >> > > >>> > > > > > > runs >> > > >>> > > > > > > > > > >>>>>> (even >> > > >>> > > > > > > > > > >>>>>>>>>> without >> > > >>> > > > > > > > > > >>>>>>>>>>>>> Java 8) when running extended >> HBase >> > > >>> tests. >> > > >>> > > > > > > > > > >>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>> -- >> > > >>> > > > > > > > > > >>>>>>>>>>>>> Jacques Nadeau >> > > >>> > > > > > > > > > >>>>>>>>>>>>> CTO and Co-Founder, Dremio >> > > >>> > > > > > > > > > >>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>> On Thu, Mar 3, 2016 at 10:02 PM, >> > Parth >> > > >>> > Chandra >> > > >>> > > < >> > > >>> > > > > > > > > > >>>>>>>> parthc@apache.org> >> > > >>> > > > > > > > > > >>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Updated list (I'll follow up >> with >> > the >> > > >>> folks >> > > >>> > > > named >> > > >>> > > > > > > here >> > > >>> > > > > > > > > > >>>>>>>>>> separately) - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Committed for 1.6 - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4384 - Query profile is >> > missing >> > > >>> > > important >> > > >>> > > > > > > > > > >>>>> information >> > > >>> > > > > > > > > > >>>>>>> on >> > > >>> > > > > > > > > > >>>>>>>>>> WebUi >> > > >>> > > > > > > > > > >>>>>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 >> > support) - >> > > >>> > Merged. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4410/pr 380 (listvector >> should >> > > >>> > > initiatlize >> > > >>> > > > > > > > > > >>>>> bits...) - >> > > >>> > > > > > > > > > >>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow custom >> > > configs >> > > >>> for >> > > >>> > > S3, >> > > >>> > > > > > > > Kerberos, >> > > >>> > > > > > > > > > >>>>>>> etc) >> > > >>> > > > > > > > > > >>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplify >> Calcite >> > > >>> parsing >> > > >>> > & >> > > >>> > > > > > planning >> > > >>> > > > > > > > > > >>>>>>>>>> integration) - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Waiting to be merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4437 (and others)/pr 394 >> > > (Operator >> > > >>> > unit >> > > >>> > > > test >> > > >>> > > > > > > > > > >>>>>>> framework). >> > > >>> > > > > > > > > > >>>>>>>>>>> Waiting >> > > >>> > > > > > > > > > >>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> be merged. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should >> > > support >> > > >>> > > inbound >> > > >>> > > > > > > > > > >>>>>>> impersonation) >> > > >>> > > > > > > > > > >>>>>>>>>>> (Jacques >> > > >>> > > > > > > > > > >>>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Drill >> > Operators >> > > >>> and >> > > >>> > > > > Functions >> > > >>> > > > > > > > > > >>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>> correctly >> > > >>> > > > > > > > > > >>>>>>>>>>>>> expose >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> their types within Calcite.) - >> > Waiting >> > > >>> for >> > > >>> > > Aman >> > > >>> > > > to >> > > >>> > > > > > > > > > >>>>> review. >> > > >>> > > > > > > > > > >>>>>>>>> (Owners: >> > > >>> > > > > > > > > > >>>>>>>>>>>>> Hsuan, >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Jinfeng, Aman, Sudheesh) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved >> client >> > > >>> > > > randomization. >> > > >>> > > > > > > > Update >> > > >>> > > > > > > > > > >>>>>>> JIRA >> > > >>> > > > > > > > > > >>>>>>>>> with >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> warnings about using the featur= e >> ) >> > > >>> (Sudheesh >> > > >>> > > to >> > > >>> > > > > > > review.) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong result= s >> > when >> > > >>> > metadata >> > > >>> > > > > cache >> > > >>> > > > > > > is >> > > >>> > > > > > > > > > >>>>>>> used..) >> > > >>> > > > > > > > > > >>>>>>>>>> (Aman >> > > >>> > > > > > > > > > >>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4069/pr 352 Enable RPC >> thread >> > > >>> offload >> > > >>> > by >> > > >>> > > > > > default >> > > >>> > > > > > > > > > >>>>>>> (Owner: >> > > >>> > > > > > > > > > >>>>>>>>>>>> Sudheesh) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Need review - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the mave= n >> > > release >> > > >>> > > > profile) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4452/pr 395 (Update Avati= ca >> > > >>> Driver to >> > > >>> > > > latest >> > > >>> > > > > > > > > > >>>>> Calcite) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4332/pr 389 (Make vector >> > > >>> comparison >> > > >>> > > order >> > > >>> > > > > > stable >> > > >>> > > > > > > > in >> > > >>> > > > > > > > > > >>>>>>> test >> > > >>> > > > > > > > > > >>>>>>>>>>>> framework) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4411/pr 381 (hash join >> > > over-memory >> > > >>> > > > > condition) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan >> should >> > > not >> > > >>> use >> > > >>> > > star >> > > >>> > > > > > > column) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4184/pr 372 (support >> variable >> > > >>> length >> > > >>> > > > decimal >> > > >>> > > > > > > > fields >> > > >>> > > > > > > > > > >>>>>> in >> > > >>> > > > > > > > > > >>>>>>>>>> parquet) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4120 - dir0 does not work >> when >> > > the >> > > >>> > > > directory >> > > >>> > > > > > > > > > >>>>>> structure >> > > >>> > > > > > > > > > >>>>>>>>>> contains >> > > >>> > > > > > > > > > >>>>>>>>>>>>> Avro >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> files - Partial patch available= . >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4203/pr 341 (fix dates >> written >> > > >>> into >> > > >>> > > > parquet >> > > >>> > > > > > > files >> > > >>> > > > > > > > > > >>>>> to >> > > >>> > > > > > > > > > >>>>>>>>> conform >> > > >>> > > > > > > > > > >>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> parquet format spec) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Not included (yet) - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-3149 - No patch available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4441 - IN operator does n= ot >> > work >> > > >>> with >> > > >>> > > Avro >> > > >>> > > > > > > reader >> > > >>> > > > > > > > - >> > > >>> > > > > > > > > > >>>>>> No >> > > >>> > > > > > > > > > >>>>>>>>> patch >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive char >> > support >> > > - >> > > >>> New >> > > >>> > > > > feature >> > > >>> > > > > > - >> > > >>> > > > > > > > > > >>>>> Needs >> > > >>> > > > > > > > > > >>>>>>> QA >> > > >>> > > > > > > > > > >>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>> Not >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> included in 1.6 >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-3623 - Limit 0 should avo= id >> > > >>> execution >> > > >>> > > when >> > > >>> > > > > > > > > > >>>>> querying a >> > > >>> > > > > > > > > > >>>>>>>> known >> > > >>> > > > > > > > > > >>>>>>>>>>>> schema. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> (Need to add limitations of >> current >> > > >>> impl). >> > > >>> > > > > Intrusive >> > > >>> > > > > > > > > > >>>>>> change; >> > > >>> > > > > > > > > > >>>>>>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> included at beginning of releas= e >> > > cycle. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4416/pr 385 (quote path >> > > separator) >> > > >>> > > (Owner: >> > > >>> > > > > > > Hanifi) >> > > >>> > > > > > > > > > >>>>> - >> > > >>> > > > > > > > > > >>>>>>>> Causes >> > > >>> > > > > > > > > > >>>>>>>>>>> leak. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Others - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-2517 - Already resolved= . >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-3688/pr 382 >> > > >>> (skip.header.line.count in >> > > >>> > > > > hive). >> > > >>> > > > > > - >> > > >>> > > > > > > > > > >>>>>> Already >> > > >>> > > > > > > > > > >>>>>>>>>> merged. >> > > >>> > > > > > > > > > >>>>>>>>>>>> PR >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> needs to be closed. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 9:44 PM, >> > Parth >> > > >>> > Chandra >> > > >>> > > < >> > > >>> > > > > > > > > > >>>>>>>> parthc@apache.org >> > > >>> > > > > > > > > > >>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>> Right. My mistake. Thanks, >> Jacques, >> > > for >> > > >>> > > > > reviewing. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 9:08 PM= , >> > > Zelaine >> > > >>> > Fong >> > > >>> > > < >> > > >>> > > > > > > > > > >>>>>>>>> zfong@maprtech.com >> > > >>> > > > > > > > > > >>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill shou= ld >> > > >>> support >> > > >>> > > > inbound >> > > >>> > > > > > > > > > >>>>>>>> impersonation) >> > > >>> > > > > > > > > > >>>>>>>>>>>>> (Sudheesh >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> Sudheesh is the fixer of >> > DRILL-4281, >> > > >>> so I >> > > >>> > > > don't >> > > >>> > > > > > > think >> > > >>> > > > > > > > > > >>>>> he >> > > >>> > > > > > > > > > >>>>>>> can >> > > >>> > > > > > > > > > >>>>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>> the >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> reviewer :). >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> -- Zelaine >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 6:30 P= M, >> > > Parth >> > > >>> > > Chandra >> > > >>> > > > < >> > > >>> > > > > > > > > > >>>>>>>>>> parthc@apache.org> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Here's an updated list with >> names >> > > of >> > > >>> > > > reviewers >> > > >>> > > > > > > > > > >>>>> added. >> > > >>> > > > > > > > > > >>>>>> If >> > > >>> > > > > > > > > > >>>>>>>>>> anyone >> > > >>> > > > > > > > > > >>>>>>>>>>>> else >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> is >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> reviewing the open PRs pleas= e >> let >> > > me >> > > >>> > know. >> > > >>> > > > Some >> > > >>> > > > > > PRs >> > > >>> > > > > > > > > > >>>>>> have >> > > >>> > > > > > > > > > >>>>>>>>>> owners >> > > >>> > > > > > > > > > >>>>>>>>>>>>> names >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> that >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> I will follow up with. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Jason, I've included your >> JIRA in >> > > the >> > > >>> > list. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Committed for 1.6 - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4384 - Query profile i= s >> > > missing >> > > >>> > > > important >> > > >>> > > > > > > > > > >>>>>>>> information >> > > >>> > > > > > > > > > >>>>>>>>> on >> > > >>> > > > > > > > > > >>>>>>>>>>>>> WebUi - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 >> > > support) >> > > >>> - >> > > >>> > > > Merged. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4410/pr 380 (listvecto= r >> > > should >> > > >>> > > > > initiatlize >> > > >>> > > > > > > > > > >>>>>>> bits...) >> > > >>> > > > > > > > > > >>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow >> custom >> > > >>> configs >> > > >>> > for >> > > >>> > > > S3, >> > > >>> > > > > > > > > > >>>>>>> Kerberos, >> > > >>> > > > > > > > > > >>>>>>>>>> etc) - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplify >> > Calcite >> > > >>> > > parsing & >> > > >>> > > > > > > > > > >>>>> planning >> > > >>> > > > > > > > > > >>>>>>>>>>>> integration) >> > > >>> > > > > > > > > > >>>>>>>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Waiting to be merged >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill >> should >> > > >>> support >> > > >>> > > > inbound >> > > >>> > > > > > > > > > >>>>>>>>> impersonation) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> (Sudheesh to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Drill >> > > >>> Operators and >> > > >>> > > > > > Functions >> > > >>> > > > > > > > > > >>>>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>>>> correctly >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> expose >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> their types within Calcite.)= - >> > > >>> Waiting >> > > >>> > for >> > > >>> > > > Aman >> > > >>> > > > > > to >> > > >>> > > > > > > > > > >>>>>>> review. >> > > >>> > > > > > > > > > >>>>>>>>>>>> (Owners: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> Hsuan, >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Jinfeng, Aman, Sudheesh) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved >> > client >> > > >>> > > > > > randomization. >> > > >>> > > > > > > > > > >>>>>>> Update >> > > >>> > > > > > > > > > >>>>>>>>> JIRA >> > > >>> > > > > > > > > > >>>>>>>>>>>> with >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> warnings about using the >> feature >> > ) >> > > >>> > > (Sudheesh >> > > >>> > > > to >> > > >>> > > > > > > > > > >>>>>> review.) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4437 (and others)/pr 3= 94 >> > > >>> (Operator >> > > >>> > > unit >> > > >>> > > > > > test >> > > >>> > > > > > > > > > >>>>>>>>> framework). >> > > >>> > > > > > > > > > >>>>>>>>>>>>> (Parth >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong >> results >> > > when >> > > >>> > > > metadata >> > > >>> > > > > > > cache >> > > >>> > > > > > > > > > >>>>>> is >> > > >>> > > > > > > > > > >>>>>>>>>> used..) >> > > >>> > > > > > > > > > >>>>>>>>>>>>> (Aman >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> review) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4416/pr 385 (quote pat= h >> > > >>> separator) >> > > >>> > > > > (Owner: >> > > >>> > > > > > > > > > >>>>>> Hanifi) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4069/pr 352 Enable RPC >> > thread >> > > >>> > offload >> > > >>> > > > by >> > > >>> > > > > > > > > > >>>>> default >> > > >>> > > > > > > > > > >>>>>>>>> (Owner: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Sudheesh) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Need review - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the >> maven >> > > >>> release >> > > >>> > > > > profile) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4452/pr 395 (Update >> Avatica >> > > >>> Driver >> > > >>> > to >> > > >>> > > > > > latest >> > > >>> > > > > > > > > > >>>>>>>> Calcite) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4332/pr 389 (Make vect= or >> > > >>> comparison >> > > >>> > > > order >> > > >>> > > > > > > > > > >>>>> stable >> > > >>> > > > > > > > > > >>>>>>> in >> > > >>> > > > > > > > > > >>>>>>>>> test >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> framework) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4411/pr 381 (hash join >> > > >>> over-memory >> > > >>> > > > > > condition) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan >> > should >> > > >>> not >> > > >>> > use >> > > >>> > > > > star >> > > >>> > > > > > > > > > >>>>>> column) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4184/pr 372 (support >> > variable >> > > >>> > length >> > > >>> > > > > > decimal >> > > >>> > > > > > > > > > >>>>>>> fields >> > > >>> > > > > > > > > > >>>>>>>> in >> > > >>> > > > > > > > > > >>>>>>>>>>>>> parquet) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4120 - dir0 does not >> work >> > > when >> > > >>> the >> > > >>> > > > > > directory >> > > >>> > > > > > > > > > >>>>>>>> structure >> > > >>> > > > > > > > > > >>>>>>>>>>>>> contains >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> Avro >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> files - Partial patch >> available. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4203/pr 341 (fix dates >> > > written >> > > >>> into >> > > >>> > > > > parquet >> > > >>> > > > > > > > > > >>>>>> files >> > > >>> > > > > > > > > > >>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>> conform >> > > >>> > > > > > > > > > >>>>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> parquet format spec) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Not included (yet) - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3149 - No patch >> available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4441 - IN operator doe= s >> not >> > > >>> work >> > > >>> > with >> > > >>> > > > > Avro >> > > >>> > > > > > > > > > >>>>>> reader >> > > >>> > > > > > > > > > >>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>> No >> > > >>> > > > > > > > > > >>>>>>>>>>> patch >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive cha= r >> > > >>> support - >> > > >>> > New >> > > >>> > > > > > feature >> > > >>> > > > > > > > > > >>>>> - >> > > >>> > > > > > > > > > >>>>>>>> Needs >> > > >>> > > > > > > > > > >>>>>>>>>> QA - >> > > >>> > > > > > > > > > >>>>>>>>>>>> Not >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> included in 1.6 >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3623 - Limit 0 should >> avoid >> > > >>> > execution >> > > >>> > > > > when >> > > >>> > > > > > > > > > >>>>>>> querying >> > > >>> > > > > > > > > > >>>>>>>> a >> > > >>> > > > > > > > > > >>>>>>>>>>> known >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> schema. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> (Need to add limitations of >> > current >> > > >>> > impl). >> > > >>> > > > > > > Intrusive >> > > >>> > > > > > > > > > >>>>>>>> change; >> > > >>> > > > > > > > > > >>>>>>>>>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> included at beginning of >> release >> > > >>> cycle. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Others - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-2517 - Already >> resolved. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3688/pr 382 >> > > >>> (skip.header.line.count >> > > >>> > > in >> > > >>> > > > > > hive). >> > > >>> > > > > > > > > > >>>>> - >> > > >>> > > > > > > > > > >>>>>>>>> Already >> > > >>> > > > > > > > > > >>>>>>>>>>>>> merged. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> PR >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> needs to be closed. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Thanks >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Parth >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 3:21 >> PM, >> > > Jason >> > > >>> > > > > Altekruse < >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> altekrusejason@gmail.com> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> I have updated the PR for t= he >> > > >>> parquet >> > > >>> > date >> > > >>> > > > > > > > > > >>>>>> corruption >> > > >>> > > > > > > > > > >>>>>>>>> issue >> > > >>> > > > > > > > > > >>>>>>>>>>> that >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> didn't >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> make it into 1.5. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > https://github.com/apache/drill/pull/341 >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > > > > > https://issues.apache.org/jira/browse/DRILL-4203 >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> If this can get reviewed, I >> > think >> > > it >> > > >>> > would >> > > >>> > > > be >> > > >>> > > > > > good >> > > >>> > > > > > > > > > >>>>>> to >> > > >>> > > > > > > > > > >>>>>>>> get >> > > >>> > > > > > > > > > >>>>>>>>>> into >> > > >>> > > > > > > > > > >>>>>>>>>>>> the >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> release. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> Any takers? >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 11:0= 7 >> PM, >> > > >>> Parth >> > > >>> > > > > Chandra < >> > > >>> > > > > > > > > > >>>>>>>>>>>> parthc@apache.org >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> I've summarized the list o= f >> > JIRs >> > > >>> below. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> The first set of pull >> requests >> > is >> > > >>> under >> > > >>> > > > > review >> > > >>> > > > > > > > > > >>>>> (or >> > > >>> > > > > > > > > > >>>>>>>> have >> > > >>> > > > > > > > > > >>>>>>>>>> some >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> reviewer >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> assigned). >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> The second set contains pu= ll >> > > >>> requests >> > > >>> > > that >> > > >>> > > > > need >> > > >>> > > > > > > > > > >>>>>>>> review. >> > > >>> > > > > > > > > > >>>>>>>>> We >> > > >>> > > > > > > > > > >>>>>>>>>>>> need >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> committers >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> to review these. Please >> > volunteer >> > > >>> or >> > > >>> > > these >> > > >>> > > > > will >> > > >>> > > > > > > > > > >>>>>> not >> > > >>> > > > > > > > > > >>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>> able >> > > >>> > > > > > > > > > >>>>>>>>>>> to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> make >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> it >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> into >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> the release. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> The third set is Jira's >> that do >> > > not >> > > >>> > have >> > > >>> > > a >> > > >>> > > > > > patch >> > > >>> > > > > > > > > > >>>>>>>> and/or >> > > >>> > > > > > > > > > >>>>>>>>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>>>>> not >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> included because they >> require >> > > >>> deeper >> > > >>> > > > > scrutiny. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> I'm hoping we can finalize >> the >> > > >>> list of >> > > >>> > > PRs >> > > >>> > > > > that >> > > >>> > > > > > > > > > >>>>>> can >> > > >>> > > > > > > > > > >>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>>> reviewed >> > > >>> > > > > > > > > > >>>>>>>>>>>>> by >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Friday >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> morning and possibly >> *finalize >> > > the >> > > >>> list >> > > >>> > > of >> > > >>> > > > > > > > > > >>>>> issues >> > > >>> > > > > > > > > > >>>>>> to >> > > >>> > > > > > > > > > >>>>>>>> be >> > > >>> > > > > > > > > > >>>>>>>>>>>> included >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> by >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> Friday >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> end of day* so please take >> some >> > > >>> time to >> > > >>> > > > > review >> > > >>> > > > > > > > > > >>>>> the >> > > >>> > > > > > > > > > >>>>>>>> PRs. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> Also note that the QA team >> has >> > > >>> offered >> > > >>> > to >> > > >>> > > > do >> > > >>> > > > > > > > > > >>>>>> sanity >> > > >>> > > > > > > > > > >>>>>>>>>> testing >> > > >>> > > > > > > > > > >>>>>>>>>>>> once >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> we >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> decide >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> on the final commit to be >> > > included, >> > > >>> > > before >> > > >>> > > > > the >> > > >>> > > > > > > > > > >>>>>>> release >> > > >>> > > > > > > > > > >>>>>>>>>>>> candidate >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> is >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> rolled >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> out, which helps with the >> > release >> > > >>> > > candidate >> > > >>> > > > > > > > > > >>>>> moving >> > > >>> > > > > > > > > > >>>>>>>>> forward >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> smoothly. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> Here's the list - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Committed for 1.6 -* >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill >> should >> > > >>> support >> > > >>> > > > > inbound >> > > >>> > > > > > > > > > >>>>>>>>>>> impersonation) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Dril= l >> > > >>> Operators >> > > >>> > and >> > > >>> > > > > > > > > > >>>>>> Functions >> > > >>> > > > > > > > > > >>>>>>>>> should >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> correctly >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> expose >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> their types within >> Calcite.) - >> > > >>> Waiting >> > > >>> > > for >> > > >>> > > > > Aman >> > > >>> > > > > > > > > > >>>>> to >> > > >>> > > > > > > > > > >>>>>>>>> review. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4313/pr 396 (Improv= ed >> > > client >> > > >>> > > > > > > > > > >>>>> randomization. >> > > >>> > > > > > > > > > >>>>>>>>> Update >> > > >>> > > > > > > > > > >>>>>>>>>>> JIRA >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> with >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> warnings about using the >> > feature >> > > ) >> > > >>> > > Sudheesh >> > > >>> > > > > to >> > > >>> > > > > > > > > > >>>>>>> review. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.= 8 >> > > >>> support) >> > > >>> > > Hanifi >> > > >>> > > > > to >> > > >>> > > > > > > > > > >>>>>>> review >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4437 (and others)/pr >> 394 >> > > >>> > (Operator >> > > >>> > > > unit >> > > >>> > > > > > > > > > >>>>> test >> > > >>> > > > > > > > > > >>>>>>>>>>> framework). >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> Parth to >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> review >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4384 - Query profile >> is >> > > >>> missing >> > > >>> > > > > important >> > > >>> > > > > > > > > > >>>>>>>>>> information >> > > >>> > > > > > > > > > >>>>>>>>>>> on >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> WebUi - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> Marked as resolved. Patch >> not >> > > >>> applied? >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Need review -* >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplif= y >> > > Calcite >> > > >>> > > > parsing & >> > > >>> > > > > > > > > > >>>>>>> planning >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> integration) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the >> > maven >> > > >>> > release >> > > >>> > > > > > > > > > >>>>> profile) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4452/pr 395 (Update >> > Avatica >> > > >>> > Driver >> > > >>> > > to >> > > >>> > > > > > > > > > >>>>> latest >> > > >>> > > > > > > > > > >>>>>>>>>> Calcite) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4332/pr 389 (Make >> vector >> > > >>> > comparison >> > > >>> > > > > order >> > > >>> > > > > > > > > > >>>>>>> stable >> > > >>> > > > > > > > > > >>>>>>>>> in >> > > >>> > > > > > > > > > >>>>>>>>>>> test >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> framework) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong >> > results >> > > >>> when >> > > >>> > > > > metadata >> > > >>> > > > > > > > > > >>>>>> cache >> > > >>> > > > > > > > > > >>>>>>>> is >> > > >>> > > > > > > > > > >>>>>>>>>>>> used..) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4416/pr 385 (quote >> path >> > > >>> > separator) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4411/pr 381 (hash jo= in >> > > >>> > over-memory >> > > >>> > > > > > > > > > >>>>>> condition) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4410/pr 380 >> (listvector >> > > >>> should >> > > >>> > > > > > initiatlize >> > > >>> > > > > > > > > > >>>>>>>>> bits...) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupSc= an >> > > >>> should not >> > > >>> > > use >> > > >>> > > > > > star >> > > >>> > > > > > > > > > >>>>>>>> column) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow >> custom >> > > >>> configs >> > > >>> > > for >> > > >>> > > > > S3, >> > > >>> > > > > > > > > > >>>>>>>>> Kerberos, >> > > >>> > > > > > > > > > >>>>>>>>>>> etc) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4184/pr 372 (support >> > > variable >> > > >>> > > length >> > > >>> > > > > > > > > > >>>>> decimal >> > > >>> > > > > > > > > > >>>>>>>>> fields >> > > >>> > > > > > > > > > >>>>>>>>>> in >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> parquet) >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4069/pr 352 Enable R= PC >> > > thread >> > > >>> > > offload >> > > >>> > > > > by >> > > >>> > > > > > > > > > >>>>>>> default >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4120 - dir0 does not >> work >> > > >>> when >> > > >>> > the >> > > >>> > > > > > > > > > >>>>> directory >> > > >>> > > > > > > > > > >>>>>>>>>> structure >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> contains >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> Avro >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> files - Partial patch >> > available. >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Not included (yet) - * >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3149 - No patch >> available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4441 - IN operator >> does >> > not >> > > >>> work >> > > >>> > > with >> > > >>> > > > > > Avro >> > > >>> > > > > > > > > > >>>>>>>> reader >> > > >>> > > > > > > > > > >>>>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>> No >> > > >>> > > > > > > > > > >>>>>>>>>>>>> patch >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> available >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive >> char >> > > >>> support - >> > > >>> > > New >> > > >>> > > > > > > > > > >>>>>> feature >> > > >>> > > > > > > > > > >>>>>>> - >> > > >>> > > > > > > > > > >>>>>>>>>> Needs >> > > >>> > > > > > > > > > >>>>>>>>>>>> QA - >> > > >>> > > > > > > > > > >>>>>>>>>>>>>> Not >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> included in 1.6 >> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3623 - Limit 0 shoul= d >> > avoid >> > > >>> > > execution >> > > >>> > > > > > when >> > > >>> > > > > > > > > > >>>>>>>>> querying >> > > >>> > > > > > > > > > >>>>>>>> >> > > >>> >> > > >> ... >> > > >> >> > > >> [Message clipped] >> > > > >> > > > >> > > > >> > > >> > >> > > --001a113549d47971cc052dbac0fc--