drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Parth Chandra <pchan...@maprtech.com>
Subject Re: Time for the 1.6 Release
Date Tue, 08 Mar 2016 02:32:37 GMT
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
> > > > > >>>>>>>>>> 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.
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 3:11 PM, Vicky Markman <
> > > > > >>>>>>>>>>>>>>>> vmarkman@maprtech.com>
> > > > > >>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>> You are welcome, Jacques.
> > > > > >>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>> Vick*y *:)
> > > > > >>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 3:06 PM, Jacques
> > > > > >>>>> Nadeau <
> > > > > >>>>>>>>>>>>>>>> jacques@dremio.com>
> > > > > >>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>> I just realized that we didn't merge the
> > > > > >>>>>> broken
> > > > > >>>>>>>>>> profile
> > > > > >>>>>>>>>>>>> patch
> > > > > >>>>>>>>>>>>>>>>> (thanks
> > > > > >>>>>>>>>>>>>>>>>>>>> Vicki). We should get it merged as well.
> > > > > >>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>> DRILL-4384
> > > > > >>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>> --
> > > > > >>>>>>>>>>>>>>>>>>>>> Jacques Nadeau
> > > > > >>>>>>>>>>>>>>>>>>>>> CTO and Co-Founder, Dremio
> > > > > >>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 10:46 AM, Jason
> > > > > >>>>>>> Altekruse <
> > > > > >>>>>>>>>>>>>>>>>>>> altekrusejason@gmail.com
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>> I should have merged this sooner but we
> > > > > >>>>> will
> > > > > >>>>>>>> need
> > > > > >>>>>>>>>> this
> > > > > >>>>>>>>>>>>> patch
> > > > > >>>>>>>>>>>>>>>>> that I
> > > > > >>>>>>>>>>>>>>>>>>> had
> > > > > >>>>>>>>>>>>>>>>>>>>>> applied to the 1.5 release branch. The
> > > > > >>>>>> change
> > > > > >>>>>>> is
> > > > > >>>>>>>>>> small
> > > > > >>>>>>>>>>>> and
> > > > > >>>>>>>>>>>>>>>> fixes
> > > > > >>>>>>>>>>>>>>>>> a
> > > > > >>>>>>>>>>>>>>>>>>>> build
> > > > > >>>>>>>>>>>>>>>>>>>>>> problem that only appears when running the
> > > > > >>>>>>> maven
> > > > > >>>>>>>>>>> release
> > > > > >>>>>>>>>>>>>>>> profile.
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>> https://github.com/apache/drill/pull/402
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 9:28 AM, Jinfeng
> > > > > >>>>> Ni <
> > > > > >>>>>>>>>>>>>>>>> jinfengni99@gmail.com>
> > > > > >>>>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>> Hi John,
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>> I think patch for DRILL-2517 has been
> > > > > >>>>>> merged
> > > > > >>>>>>>> to
> > > > > >>>>>>>>>> the
> > > > > >>>>>>>>>>>>> apache
> > > > > >>>>>>>>>>>>>>>>> master
> > > > > >>>>>>>>>>>>>>>>>>>>>>> branch. Have you tried your query on the
> > > > > >>>>>>>> latest
> > > > > >>>>>>>>>>> master
> > > > > >>>>>>>>>>>>>>>> branch?
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>> In DRILL-2517, I posted some performance
> > > > > >>>>>>>> number
> > > > > >>>>>>>>>> for
> > > > > >>>>>>>>>>>> 117k
> > > > > >>>>>>>>>>>>>>>> small
> > > > > >>>>>>>>>>>>>>>>>>>> parquet
> > > > > >>>>>>>>>>>>>>>>>>>>>>> files. The patch did show improvement.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>> Before DRILL-3996 is resolved, for now
> > > > > >>>>> if
> > > > > >>>>>>> your
> > > > > >>>>>>>>>> query
> > > > > >>>>>>>>>>>>>> relies
> > > > > >>>>>>>>>>>>>>>> on
> > > > > >>>>>>>>>>>>>>>>>>> filter
> > > > > >>>>>>>>>>>>>>>>>>>>>>> pushdown logic to push partitioning
> > > > > >>>>> filter
> > > > > >>>>>>>>> first,
> > > > > >>>>>>>>>>> then
> > > > > >>>>>>>>>>>>> the
> > > > > >>>>>>>>>>>>>>>>> patch
> > > > > >>>>>>>>>>>>>>>>>>> for
> > > > > >>>>>>>>>>>>>>>>>>>>>>> DRILL-2517 will not help.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 4:23 AM, John
> > > > > >>>>>>> Omernik <
> > > > > >>>>>>>>>>>>>>>> john@omernik.com
> > > > > >>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> I'd like to request drill-2517 be
> > > > > >>>>> added
> > > > > >>>>>>> as a
> > > > > >>>>>>>>>>> bandaid
> > > > > >>>>>>>>>>>>> for
> > > > > >>>>>>>>>>>>>>>> the
> > > > > >>>>>>>>>>>>>>>>>>>> planning
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> issues when there are lots of
> > > > > >>>>>> directories
> > > > > >>>>>>> of
> > > > > >>>>>>>>>>> parquet
> > > > > >>>>>>>>>>>>>>>> files.
> > > > > >>>>>>>>>>>>>>>>>>> This
> > > > > >>>>>>>>>>>>>>>>>>>>>> issue
> > > > > >>>>>>>>>>>>>>>>>>>>>>> is
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> really hurting drill adoption for me
> > > > > >>>>> in
> > > > > >>>>>> my
> > > > > >>>>>>>>> org.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> Thanks,  John
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> On Tuesday, March 1, 2016, Edmon
> > > > > >>>>> Begoli
> > > > > >>>>>> <
> > > > > >>>>>>>>>>>>>>>> ebegoli@gmail.com>
> > > > > >>>>>>>>>>>>>>>>>>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> May I please ask to give this issue
> > > > > >>>>> the
> > > > > >>>>>>>>>> attention
> > > > > >>>>>>>>>>>> for
> > > > > >>>>>>>>>>>>>>>> 1.6:
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>
> > > > > >>>>>>>>>>
> > > > > >>>>>>>
> > > > > >>>
> > > >
> https://issues.apache.org/jira/plugins/servlet/mobile#issue/DRILL-3149
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> I will try to suggest a patch. Given
> > > > > >>>>> my
> > > > > >>>>>>>> time
> > > > > >>>>>>>>>>>>>> constraints
> > > > > >>>>>>>>>>>>>>>> I
> > > > > >>>>>>>>>>>>>>>>>> might
> > > > > >>>>>>>>>>>>>>>>>>>> not
> > > > > >>>>>>>>>>>>>>>>>>>>>> be
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> able to submit a complete, unit
> > > > > >>>>> tested
> > > > > >>>>>>> code
> > > > > >>>>>>>>> but
> > > > > >>>>>>>>>>> at
> > > > > >>>>>>>>>>>> I
> > > > > >>>>>>>>>>>>>>>> least I
> > > > > >>>>>>>>>>>>>>>>>>> will
> > > > > >>>>>>>>>>>>>>>>>>>>> try
> > > > > >>>>>>>>>>>>>>>>>>>>>> to
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> submit a snippet that will help with
> > > > > >>>>>>> fixing
> > > > > >>>>>>>>> it
> > > > > >>>>>>>>>> up
> > > > > >>>>>>>>>>>> (I
> > > > > >>>>>>>>>>>>>>>> think
> > > > > >>>>>>>>>>>>>>>>> we
> > > > > >>>>>>>>>>>>>>>>>>> just
> > > > > >>>>>>>>>>>>>>>>>>>>>> need
> > > > > >>>>>>>>>>>>>>>>>>>>>>> to
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> do a look ahead byte lookup to ensure
> > > > > >>>>>>> that
> > > > > >>>>>>>> it
> > > > > >>>>>>>>>> is
> > > > > >>>>>>>>>>>> not
> > > > > >>>>>>>>>>>>>>>> \r\n.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> On Tuesday, March 1, 2016, Jacques
> > > > > >>>>>>> Nadeau <
> > > > > >>>>>>>>>>>>>>>>> jacques@dremio.com
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> <javascript:;>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> It seems like a stretch to include
> > > > > >>>>>>>>> DRILL-3623
> > > > > >>>>>>>>>>>> right
> > > > > >>>>>>>>>>>>>>>> before
> > > > > >>>>>>>>>>>>>>>>>> the
> > > > > >>>>>>>>>>>>>>>>>>>>>>> release.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> This is a pretty fundamental change
> > > > > >>>>>>> that
> > > > > >>>>>>>>>> seems
> > > > > >>>>>>>>>>>> like
> > > > > >>>>>>>>>>>>>> it
> > > > > >>>>>>>>>>>>>>>>>> should
> > > > > >>>>>>>>>>>>>>>>>>>> soak
> > > > > >>>>>>>>>>>>>>>>>>>>>>> for a
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> bit of time before we release. If
> > > > > >>>>> we
> > > > > >>>>>>> want
> > > > > >>>>>>>>> to
> > > > > >>>>>>>>>>>>> include,
> > > > > >>>>>>>>>>>>>>>> I'd
> > > > > >>>>>>>>>>>>>>>>>>>> suggest
> > > > > >>>>>>>>>>>>>>>>>>>>>>> that we
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> disable the functionality by
> > > > > >>>>> default
> > > > > >>>>>>> and
> > > > > >>>>>>>>>>> consider
> > > > > >>>>>>>>>>>>> it
> > > > > >>>>>>>>>>>>>>>>>>>> experimental.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> I'll propose a few other patches
> > > > > >>>>> for
> > > > > >>>>>>>>>> inclusion
> > > > > >>>>>>>>>>>>>> shortly.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> --
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> Jacques Nadeau
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> CTO and Co-Founder, Dremio
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> On Tue, Mar 1, 2016 at 5:04 PM,
> > > > > >>>>> Parth
> > > > > >>>>>>>>>> Chandra <
> > > > > >>>>>>>>>>>>>>>>>>>> parthc@apache.org
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>> <javascript:;>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> <javascript:;>> wrote:
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> Hello everyone,
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> It's time to start looking into
> > > > > >>>>>> the
> > > > > >>>>>>>> 1.6
> > > > > >>>>>>>>>>>>> release.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> Can all the folks working on
> > > > > >>>>> open
> > > > > >>>>>>>>> issues
> > > > > >>>>>>>>>>> let
> > > > > >>>>>>>>>>>> me
> > > > > >>>>>>>>>>>>>>>> know
> > > > > >>>>>>>>>>>>>>>>> if
> > > > > >>>>>>>>>>>>>>>>>>>> there
> > > > > >>>>>>>>>>>>>>>>>>>>>> are
> > > > > >>>>>>>>>>>>>>>>>>>>>>> any
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> JIRAs you would like to get into
> > > > > >>>>>> the
> > > > > >>>>>>>>>> release?
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> I know of the following -
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> *DRILL-4281 *- Drill should
> > > > > >>>>> support
> > > > > >>>>>>>>> inbound
> > > > > >>>>>>>>>>>>>>>>> impersonation.
> > > > > >>>>>>>>>>>>>>>>>>>> Pull
> > > > > >>>>>>>>>>>>>>>>>>>>>>> request
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> expected today. C++ client to be
> > > > > >>>>>>>> tested.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> *DRILL-4372* - Drill Operators
> > > > > >>>>> and
> > > > > >>>>>>>>>> Functions
> > > > > >>>>>>>>>>>>> should
> > > > > >>>>>>>>>>>>>>>>>>> correctly
> > > > > >>>>>>>>>>>>>>>>>>>>>> expose
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> their
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> types within Calcite. Waiting for
> > > > > >>>>>>>> review.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> *DRILL-3623* - Limit 0 should
> > > > > >>>>> avoid
> > > > > >>>>>>>>>> execution
> > > > > >>>>>>>>>>>>> when
> > > > > >>>>>>>>>>>>>>>>>> querying
> > > > > >>>>>>>>>>>>>>>>>>> a
> > > > > >>>>>>>>>>>>>>>>>>>>>> known
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> schema.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> Pull request expected today. Need
> > > > > >>>>>> to
> > > > > >>>>>>>> add
> > > > > >>>>>>>>>>>>>> limitations
> > > > > >>>>>>>>>>>>>>>> of
> > > > > >>>>>>>>>>>>>>>>>>>> current
> > > > > >>>>>>>>>>>>>>>>>>>>>>> impl to
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>> the
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> JIRA. Review needed.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> *DRILL-4313* - Improved client
> > > > > >>>>>>>>>> randomization.
> > > > > >>>>>>>>>>>>>> Update
> > > > > >>>>>>>>>>>>>>>>> JIRA
> > > > > >>>>>>>>>>>>>>>>>>> with
> > > > > >>>>>>>>>>>>>>>>>>>>>>> warnings
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> about using the feature. Waiting
> > > > > >>>>>> for
> > > > > >>>>>>>>>> review.
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> Thanks
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>> Parth
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> --
> > > > > >>>>>>>>>>>>>>>>>>>>>>>> Sent from my iThing
> > > > > >>>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>>
> > > > > >>>>>>>>>>>>
> > > > > >>>>>>>>>>>
> > > > > >>>>>>>>>>
> > > > > >>>>>>>>>
> > > > > >>>>>>>>
> > > > > >>>>>>>
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>> --
> > > > > >>>>>>
> > > > > >>>>>> Abdelhakim Deneche
> > > > > >>>>>>
> > > > > >>>>>> Software Engineer
> > > > > >>>>>>
> > > > > >>>>>> <http://www.mapr.com/>
> > > > > >>>>>>
> > > > > >>>>>>
> > > > > >>>>>> Now Available - Free Hadoop On-Demand Training
> > > > > >>>>>> <
> > > > > >>>>>>
> > > > > >>>>>
> > > > > >>>
> > > > > >>
> > > > >
> > > >
> > >
> >
> http://www.mapr.com/training?utm_source=Email&utm_medium=Signature&utm_campaign=Free%20available
> > > > > >>>>>>>
> > > > > >>>>>>
> > > > > >>>>>
> > > > > >>>
> > > > > >>>
> > > > > >>
> > > > >
> > > > >
> > > >
> > >
> >
>

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