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 <[email protected]> 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 schema > 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án Baxter <[email protected]> > 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án > > > > On Thu, Mar 10, 2016 at 3:50 AM, Parth Chandra <[email protected]> > wrote: > > > > > Hi everyone, > > > > > > Just a note to update everyone that the QA team is testing out the > > 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 <[email protected]> > 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 <[email protected]> > > wrote: > > > > > > > >> OK, let's leave it out then. > > > >> > > > >> On Tue, Mar 8, 2016 at 9:25 AM, Jason Altekruse < > > > [email protected] > > > >> > wrote: > > > >> > > > >>> To be honest I was expecting a longer review cycle so I hadn't run > > 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 the > > > >>> 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 <[email protected]> > > > wrote: > > > >>> > > > >>> > Sounds good Jason. Let's finalize this in the hangout. > > > >>> > Do you have the expected plans for the failing tests? If so can > 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 < > > [email protected] > > > > > > > >>> > wrote: > > > >>> > > > > >>> > > We should update the expected results (i.e. the expected query > > 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 < > > > >>> > [email protected]> > > > >>> > > 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 just > > going > > > >>> to be > > > >>> > > to > > > >>> > > > mark the tests as failing: > > > >>> > > > > > > >>> > > > > > > >>> > > > > > >>> > > > > >>> > > > > > > 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 > > > >>> > > > > > > >>> > > > I will just go ahead and merge the other two fixes as I had a > > > clean > > > >>> > unit > > > >>> > > > test run and neither of the patches will have any impact on > 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, so > > 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 < > > > >>> [email protected]> > > > >>> > > > 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 in > > test > > > >>> > > framework) > > > >>> > > > > (?) > > > >>> > > > > DRILL-4474/pr 406 (Ensure that ConvertCountToDirectScan > only > > > >>> pushes > > > >>> > > > through > > > >>> > > > > project when project is trivial.) - Tests need to be fixed > > > >>> (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 should > > > >>> correctly > > > >>> > > > expose > > > >>> > > > > their types within Calcite.) Pushed out because of test > > > failures > > > >>> > > > > > > > >>> > > > > Jason, at the moment it looks like you've volunteered for > too > > > >>> much. > > > >>> > Can > > > >>> > > > we > > > >>> > > > > get other folks to help out? > > > >>> > > > > > > > >>> > > > > Thanks > > > >>> > > > > > > > >>> > > > > Parth > > > >>> > > > > > > > >>> > > > > > > > >>> > > > > On Mon, Mar 7, 2016 at 5:32 PM, Jason Altekruse < > > > >>> > > > [email protected]> > > > >>> > > > > wrote: > > > >>> > > > > > > > >>> > > > > > 4474 is actually revealing a few invalid tests in the > > > >>> Regression > > > >>> > > suite > > > >>> > > > > that > > > >>> > > > > > test for the current incorrect plans. The fix should be > > > >>> 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 < > > > >>> [email protected] > > > >>> > > > > > >>> > > > > 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 < > > > >>> > > > > > [email protected]> > > > >>> > > > > > > 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 version > > also > > > >>> does > > > >>> > > not > > > >>> > > > > make > > > >>> > > > > > > any > > > >>> > > > > > > > behavior changes, so I think it also makes sense to > > keep > > > >>> it off > > > >>> > > the > > > >>> > > > > > > release > > > >>> > > > > > > > branch. > > > >>> > > > > > > > > > > >>> > > > > > > > I will be merging the fix for 4375 the maven release > > > >>> 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 < > > > >>> > > > > > > [email protected] > > > >>> > > > > > > > > > > > >>> > > > > > > > wrote: > > > >>> > > > > > > > > > > >>> > > > > > > > > WebUI profile issue: this is a regression cause by > > > >>> > refactoring > > > >>> > > of > > > >>> > > > > > > Calcite > > > >>> > > > > > > > > integration code (DRILL-4465) which sets the text > > plan > > > >>> only > > > >>> > if > > > >>> > > > > debug > > > >>> > > > > > is > > > >>> > > > > > > > > enabled. Will submit a patch soon. > > > >>> > > > > > > > > > > > >>> > > > > > > > > On Mon, Mar 7, 2016 at 1:29 PM, Sudheesh Katkam < > > > >>> > > > > > [email protected]> > > > >>> > > > > > > > > wrote: > > > >>> > > > > > > > > > > > >>> > > > > > > > > > Thanks for clarifying Jacques. > > > >>> > > > > > > > > > > > > >>> > > > > > > > > > I haven’t looked into the fix for DRILL-4384; I > > > >>> reopened it > > > >>> > > > > because > > > >>> > > > > > > the > > > >>> > > > > > > > > > description mentioned “visualized plan” section > is > > > >>> (also) > > > >>> > > > empty. > > > >>> > > > > > > > > > > > > >>> > > > > > > > > > Thank you, > > > >>> > > > > > > > > > Sudheesh > > > >>> > > > > > > > > > > > > >>> > > > > > > > > > > On Mar 7, 2016, at 1:08 PM, Jacques Nadeau < > > > >>> > > > [email protected] > > > >>> > > > > > > > > >>> > > > > > > > wrote: > > > >>> > > > > > > > > > > > > > >>> > > > > > > > > > > The new bug (currently filed under DRILL-4384) > > is a > > > >>> > > > completely > > > >>> > > > > > > > > different > > > >>> > > > > > > > > > > bug than the original (original one has to do > > 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 > < > > > >>> > > > > > > > [email protected]> > > > >>> > > > > > > > > > > wrote: > > > >>> > > > > > > > > > > > > > >>> > > > > > > > > > >> DRILL-4384 is a blocker for the release though > > > >>> > > > > > > > > > >> > > > >>> > > > > > > > > > >> On Mon, Mar 7, 2016 at 12:01 PM, Sudheesh > > Katkam < > > > >>> > > > > > > > > [email protected]> > > > >>> > > > > > > > > > >> 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 < > > > >>> > > > > > > > > > [email protected] > > > >>> > > > > > > > > > >>> > > > >>> > > > > > > > > > >>> wrote: > > > >>> > > > > > > > > > >>>> > > > >>> > > > > > > > > > >>>> I don't know if there are any specific time > > > >>> > constraints > > > >>> > > > for > > > >>> > > > > > > > getting > > > >>> > > > > > > > > > out > > > >>> > > > > > > > > > >>> the > > > >>> > > > > > > > > > >>>> release, but I'm inclined to go with Vicky > 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 > > < > > > >>> > > > > > > [email protected] > > > >>> > > > > > > > > > > > >>> > > > > > > > > > >>> wrote: > > > >>> > > > > > > > > > >>>> > > > >>> > > > > > > > > > >>>>> Hakim, > > > >>> > > > > > > > > > >>>>> > > > >>> > > > > > > > > > >>>>> Yes, we'll include this in the release. > > > >>> > > > > > > > > > >>>>> > > > >>> > > > > > > > > > >>>>> -- Zelaine > > > >>> > > > > > > > > > >>>>> > > > >>> > > > > > > > > > >>>>> On Mon, Mar 7, 2016 at 9:31 AM, Abdel Hakim > > > >>> Deneche < > > > >>> > > > > > > > > > >>> [email protected] > > > >>> > > > > > > > > > >>>>>> > > > >>> > > > > > > > > > >>>>> 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 > > > < > > > >>> > > > > > > > [email protected]> > > > >>> > > > > > > > > > >>> wrote: > > > >>> > > > > > > > > > >>>>>> > > > >>> > > > > > > > > > >>>>>>> Hi guys, > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> I'm still waiting for the following to be > > > >>> > > > reviewed/merged > > > >>> > > > > > by > > > >>> > > > > > > > > today. > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> DRILL-4437 (and others)/pr 394 (Operator > > unit > > > >>> test > > > >>> > > > > > > framework). > > > >>> > > > > > > > > > >> Waiting > > > >>> > > > > > > > > > >>>>> to > > > >>> > > > > > > > > > >>>>>>> be merged (Jason) > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> DRILL-4372/pr 377(?) (Drill Operators 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 > > > >>> > > > > - > > > >>> > > > > > > > patch > > > >>> > > > > > > > > > >>>>>> reviewed. > > > >>> > > > > > > > > > >>>>>>> No +1) > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> DRILL-4375/pr 402 (Fix the maven release > > > >>> profile) - > > > >>> > > > > (Jason > > > >>> > > > > > - > > > >>> > > > > > > > > patch > > > >>> > > > > > > > > > >>>>>>> reviewed. Ready to merge?) > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> Thanks > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> Parth > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>> On Sun, Mar 6, 2016 at 12:01 PM, Aditya < > > > >>> > > > > > > > [email protected] > > > >>> > > > > > > > > > > > > >>> > > > > > > > > > >>>>> wrote: > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>>> DRILL-4375/pr 402 - reviewed. > > > >>> > > > > > > > > > >>>>>>>> > > > >>> > > > > > > > > > >>>>>>>> On Sun, Mar 6, 2016 at 12:48 AM, Stefán > > > >>> Baxter < > > > >>> > > > > > > > > > >>>>>>> [email protected]> > > > >>> > > > > > > > > > >>>>>>>> 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, Parth > > > >>> Chandra < > > > >>> > > > > > > > > [email protected] > > > >>> > > > > > > > > > > > > > >>> > > > > > > > > > >>>>>>> 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 under > > > 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 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) - > > > >>> > > > > > > > > > >>>>>>>>>> Merged > > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4467 - Baselines need updating. > > > >>> > > > > > > > > > >>>>>>>>>> DRILL-4449/pr 389 (Wrong results when > > > >>> 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 when > > > >>> 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 Operators > > 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 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) > > > >>> > > > > > > > > > >>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>> On Fri, Mar 4, 2016 at 6:24 PM, > Jacques > > > >>> Nadeau < > > > >>> > > > > > > > > > >>>>> [email protected] > > > >>> > > > > > > > > > >>>>>>> > > > >>> > > > > > > > > > >>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>> Awesome. Thanks Chun! > > > >>> > > > > > > > > > >>>>>>>>>>> On Mar 4, 2016 5:51 PM, "Chun Chang" > < > > > >>> > > > > > > [email protected]> > > > >>> > > > > > > > > > >>>>>> 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 > > > >>> > > < > > > >>> > > > > > > > > > >>>>>>>> [email protected]> > > > >>> > > > > > > > > > >>>>>>>>>>>> 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 > > > >>> > > < > > > >>> > > > > > > > > > >>>>>>>> [email protected]> > > > >>> > > > > > > > > > >>>>>>>>>>>> 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 feature ) > > > >>> (Sudheesh > > > >>> > > to > > > >>> > > > > > > review.) > > > >>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong results > > 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 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-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 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 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. > > > >>> > > > > > > > > > >>>>>>>>>>>>>> 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 > > > >>> > > < > > > >>> > > > > > > > > > >>>>>>>> [email protected] > > > >>> > > > > > > > > > >>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>> Right. My mistake. Thanks, > Jacques, > > > for > > > >>> > > > > reviewing. > > > >>> > > > > > > > > > >>>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 9:08 PM, > > > Zelaine > > > >>> > Fong > > > >>> > > < > > > >>> > > > > > > > > > >>>>>>>>> [email protected] > > > >>> > > > > > > > > > >>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should > > > >>> 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 PM, > > > Parth > > > >>> > > Chandra > > > >>> > > > < > > > >>> > > > > > > > > > >>>>>>>>>> [email protected]> > > > >>> > > > > > > > > > >>>>>>>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> Here's an updated list with > names > > > of > > > >>> > > > reviewers > > > >>> > > > > > > > > > >>>>> added. > > > >>> > > > > > > > > > >>>>>> If > > > >>> > > > > > > > > > >>>>>>>>>> anyone > > > >>> > > > > > > > > > >>>>>>>>>>>> else > > > >>> > > > > > > > > > >>>>>>>>>>>>>> is > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> reviewing the open PRs please > 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 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-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 394 > > > >>> (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 path > > > >>> 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 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 > 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 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 < > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> [email protected]> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> I have updated the PR for the > > > >>> 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:07 > PM, > > > >>> Parth > > > >>> > > > > Chandra < > > > >>> > > > > > > > > > >>>>>>>>>>>> [email protected] > > > >>> > > > > > > > > > >>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> I've summarized the list of > > JIRs > > > >>> below. > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> The first set of pull > requests > > is > > > >>> under > > > >>> > > > > review > > > >>> > > > > > > > > > >>>>> (or > > > >>> > > > > > > > > > >>>>>>>> have > > > >>> > > > > > > > > > >>>>>>>>>> some > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> reviewer > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> assigned). > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> The second set contains pull > > > >>> 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(?) (Drill > > > >>> Operators > > > >>> > and > > > >>> > > > > > > > > > >>>>>> Functions > > > >>> > > > > > > > > > >>>>>>>>> should > > > >>> > > > > > > > > > >>>>>>>>>>>>>> correctly > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>> expose > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> their types within Calcite.) > - > > > >>> Waiting > > > >>> > > for > > > >>> > > > > Aman > > > >>> > > > > > > > > > >>>>> to > > > >>> > > > > > > > > > >>>>>>>>> review. > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved > > > 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 (Simplify > > > 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 join > > > >>> > over-memory > > > >>> > > > > > > > > > >>>>>> condition) > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4410/pr 380 (listvector > > > >>> should > > > >>> > > > > > initiatlize > > > >>> > > > > > > > > > >>>>>>>>> bits...) > > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan > > > >>> 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 RPC > > > 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 should > > avoid > > > >>> > > execution > > > >>> > > > > > when > > > >>> > > > > > > > > > >>>>>>>>> querying > > > >>> > > > > > > > > > >>>>>>>> > > > >>> > > > >> ... > > > >> > > > >> [Message clipped] > > > > > > > > > > > > > > > > > >
