drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Parth Chandra <par...@apache.org>
Subject Re: Time for the 1.6 Release
Date Fri, 11 Mar 2016 00:09:59 GMT
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 <pchandra@maprtech.com>
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 <jason@dremio.com> 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 <stefan@activitystream.com
>> >
>> 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 <parthc@apache.org>
>> 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 <parthc@apache.org>
>> 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 <parthc@apache.org>
>> > 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 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 <parthc@apache.org>
>> > > 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 <
>> > kfaraaz@maprtech.com
>> > > >
>> > > >>> > 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 <
>> > > >>> > 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 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 <
>> > > >>> 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 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 <
>> > > >>> > > > 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 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 <
>> > > >>> 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 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 <
>> > > >>> > > > > > > venki.korukanti@gmail.com
>> > > >>> > > > > > > > >
>> > > >>> > > > > > > > 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 <
>> > > >>> > > > > > skatkam@maprtech.com>
>> > > >>> > > > > > > > > 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 <
>> > > >>> > > > jacques@dremio.com
>> > > >>> > > > > >
>> > > >>> > > > > > > > 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 <
>> > > >>> > > > > > > > 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 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
>> > <
>> > > >>> > > > > > > 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 (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
>> <
>> > > >>> > > > > > > > adityakishore@gmail.com
>> > > >>> > > > > > > > > >
>> > > >>> > > > > > > > > > >>>>> wrote:
>> > > >>> > > > > > > > > > >>>>>>>
>> > > >>> > > > > > > > > > >>>>>>>> DRILL-4375/pr 402 - reviewed.
>> > > >>> > > > > > > > > > >>>>>>>>
>> > > >>> > > > > > > > > > >>>>>>>> On Sun, Mar 6, 2016 at 12:48 AM, Stefán
>> > > >>> 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, Parth
>> > > >>> 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 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 <
>> > > >>> > > > > > > > > > >>>>> 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 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
>> > > >>> > > <
>> > > >>> > > > > > > > > > >>>>>>>> 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 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
>> > > >>> > > > <
>> > > >>> > > > > > > > > > >>>>>>>>>> parthc@apache.org>
>> > > >>> > > > > > > > > > >>>>>>>>>>>>>> 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 <
>> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>> altekrusejason@gmail.com>
>> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> 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 <
>> > > >>> > > > > > > > > > >>>>>>>>>>>> parthc@apache.org
>> > > >>> > > > > > > > > > >>>>>>>>>>>>>>
>> > > >>> > > > > > > > > > >>>>>>>>>>>>>>>>> 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]
>> > > >
>> > > >
>> > > >
>> > >
>> >
>>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message