To be honest I was expecting a longer review cycle so I hadn't run the unit tests before posting it for review. There were only very minor functional changes, so I wasn't thinking it would be an issue, and I was anticipating having to update the patch before merging it. I could update the test that is failing but I don't see much sense in trying to get it into the release because it only introduces new tests and some small core refactoring.
I'm all for getting it merged so everyone can start using it, I just think it doesn't really matter if it happens on the release branch or back on master once we cut a release branch. I would rather try to focus on getting the Avro issues resolved, which is what I'm working on right now. - Jason On Tue, Mar 8, 2016 at 8:58 AM, Parth Chandra <[email protected]> wrote: > Sounds good Jason. Let's finalize this in the hangout. > Do you have the expected plans for the failing tests? If so can you update > those and put in a pull request and we'll merge and run the tests. > Any reason for the operator test framework to be punted? You have a +1 to > merge it. > > > > On Mon, Mar 7, 2016 at 9:33 PM, Khurram Faraaz <[email protected]> > wrote: > > > We should update the expected results (i.e. the expected query plan in > this > > case) and not mark them as Failing. We do not have a Failing test > directory > > today. > > > > If we are sure the expected query plans for these tests are incorrect > > (today), we should go ahead and update the expected results files with > the > > correct query plans. > > > > Functional/ctas/ctas_auto_partition/existing_partition_ > > pruning/hierarchical/plan/count2_MD-185.q > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_2.q > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_4.q > > > > Thanks, > > Khurram > > > > On Tue, Mar 8, 2016 at 8:58 AM, Jason Altekruse < > [email protected]> > > wrote: > > > > > The cutoff time sounds good to me. I don't think I'm too overburdened, > I > > > mostly tried putting together a merge branch today and had to kick a > few > > > things out as I found issues. > > > > > > As we are not running the full regression cluster at Dremio, it would > be > > > helpful if someone could merge Jacques patch (DRILL-4474) after running > > the > > > complete set of tests. Could someone at MapR pick up the test updates > and > > > merging this patch? > > > > > > These are the failures I was seeing so far, my fix was just going to be > > to > > > mark the tests as failing: > > > > > > > > > Functional/ctas/ctas_auto_partition/existing_partition_pruning/hierarchical/plan/count2_MD-185.q > > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_2.q > > > Functional/ctas/ctas_auto_partition/general/plan/drill3947_4.q > > > > > > I will just go ahead and merge the other two fixes as I had a clean > unit > > > test run and neither of the patches will have any impact on the > > regression > > > tests. One is a build fix, that only changes behavior when running the > > > release profile and the other is a unit test fix. > > > > > > I had punted the operator unit tests out of the release, so those can > be > > > removed from your list. > > > > > > I will start looking back at both of the Avro issues right after > merging. > > > > > > On Mon, Mar 7, 2016 at 6:32 PM, Parth Chandra <[email protected]> > > > wrote: > > > > > > > The train is about to leave :). I'd like to propose a cutoff for > > > tomorrow 5 > > > > pm. > > > > (This will be a full week after the initial mail so plenty of time > for > > > > everyone to get their issues aired.) > > > > > > > > Here are the open items (that I am aware of) - > > > > > > > > Waiting to be merged > > > > DRILL-4483/pr 411 (Fix text plan regression in query profiles) > (Venki) > > > > DRILL-4437 (and others)/pr 394 (Operator unit test framework). > (Jason) > > > > DRILL-4332/pr 389 (Make vector comparison order stable in test > > framework) > > > > (?) > > > > DRILL-4474/pr 406 (Ensure that ConvertCountToDirectScan only pushes > > > through > > > > project when project is trivial.) - Tests need to be fixed (Jason) > > > > > > > > Need review - > > > > DRILL-4375/pr 402 (Fix the maven release profile) - (Jason - patch > > > > reviewed. Merge) > > > > DRILL-4486/pr 412 (Fix expression serialization escaping) (Need are > > > > reviewer) > > > > DRILL-4485 - Update MapR profile to use MapR 5.1.0 (Parth/Aditya) > > > > > > > > Regression > > > > DRILL-4482 - Avro no longer selects data correctly from a > > sub-structure. > > > No > > > > patch available. (Jason) > > > > > > > > Dropped > > > > DRILL-4372/pr 377(?) (Drill Operators and Functions should correctly > > > expose > > > > their types within Calcite.) Pushed out because of test failures > > > > > > > > Jason, at the moment it looks like you've volunteered for too much. > Can > > > we > > > > get other folks to help out? > > > > > > > > Thanks > > > > > > > > Parth > > > > > > > > > > > > On Mon, Mar 7, 2016 at 5:32 PM, Jason Altekruse < > > > [email protected]> > > > > wrote: > > > > > > > > > 4474 is actually revealing a few invalid tests in the Regression > > suite > > > > that > > > > > test for the current incorrect plans. The fix should be included in > > the > > > > > release, but I will post a PR on the regression suite to update the > > > tests > > > > > before I push it. > > > > > > > > > > On Mon, Mar 7, 2016 at 4:44 PM, Steven Phillips <[email protected] > > > > > > wrote: > > > > > > > > > > > DRILL-4486 is a pretty simple fix. Without it, currently some > regex > > > > > queries > > > > > > will fail. > > > > > > > > > > > > I think we should include it in the release. > > > > > > > > > > > > > > > > > > https://github.com/apache/drill/pull/412 > > > > > > > > > > > > On Mon, Mar 7, 2016 at 2:15 PM, Jason Altekruse < > > > > > [email protected]> > > > > > > wrote: > > > > > > > > > > > > > There is a small test issue with some of the refactoring that > > > > > accompanied > > > > > > > the operator unit tests. These don't change any user-facing > > > behavior, > > > > > so > > > > > > I > > > > > > > don't think it really needs to get into the release. I will be > > > > working > > > > > to > > > > > > > merge them into master after we cut the release branch. > > > > > > > > > > > > > > The change to update the avatica JDBC driver version also does > > not > > > > make > > > > > > any > > > > > > > behavior changes, so I think it also makes sense to keep it off > > the > > > > > > release > > > > > > > branch. > > > > > > > > > > > > > > I will be merging the fix for 4375 the maven release profile, > > 4474 > > > > > wrong > > > > > > > results with incorrect creation of DirectScan and 4332 fixing a > > > unit > > > > > test > > > > > > > to work in JDK 8, after another test run. > > > > > > > > > > > > > > On Mon, Mar 7, 2016 at 1:53 PM, Venki Korukanti < > > > > > > [email protected] > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > WebUI profile issue: this is a regression cause by > refactoring > > of > > > > > > Calcite > > > > > > > > integration code (DRILL-4465) which sets the text plan only > if > > > > debug > > > > > is > > > > > > > > enabled. Will submit a patch soon. > > > > > > > > > > > > > > > > On Mon, Mar 7, 2016 at 1:29 PM, Sudheesh Katkam < > > > > > [email protected]> > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Thanks for clarifying Jacques. > > > > > > > > > > > > > > > > > > I haven’t looked into the fix for DRILL-4384; I reopened it > > > > because > > > > > > the > > > > > > > > > description mentioned “visualized plan” section is (also) > > > empty. > > > > > > > > > > > > > > > > > > Thank you, > > > > > > > > > Sudheesh > > > > > > > > > > > > > > > > > > > On Mar 7, 2016, at 1:08 PM, Jacques Nadeau < > > > [email protected] > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > The new bug (currently filed under DRILL-4384) is a > > > completely > > > > > > > > different > > > > > > > > > > bug than the original (original one has to do with > profile > > > > > metrics, > > > > > > > > this > > > > > > > > > > has to do with plan text). I try to look at it tonight if > > > noone > > > > > can > > > > > > > get > > > > > > > > > to > > > > > > > > > > it sooner. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Jacques Nadeau > > > > > > > > > > CTO and Co-Founder, Dremio > > > > > > > > > > > > > > > > > > > > On Mon, Mar 7, 2016 at 12:37 PM, Parth Chandra < > > > > > > > [email protected]> > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > >> DRILL-4384 is a blocker for the release though > > > > > > > > > >> > > > > > > > > > >> On Mon, Mar 7, 2016 at 12:01 PM, Sudheesh Katkam < > > > > > > > > [email protected]> > > > > > > > > > >> wrote: > > > > > > > > > >> > > > > > > > > > >>> I reopened DRILL-4384 < > > > > > > > > > https://issues.apache.org/jira/browse/DRILL-4384> > > > > > > > > > >>> (blocker); it is assigned to Jacques. > > > > > > > > > >>> > > > > > > > > > >>> On the latest master, the visualized and physical plan > > tabs > > > > on > > > > > > web > > > > > > > UI > > > > > > > > > are > > > > > > > > > >>> empty. > > > > > > > > > >>> > > > > > > > > > >>> Thank you, > > > > > > > > > >>> Sudheesh > > > > > > > > > >>> > > > > > > > > > >>>> On Mar 7, 2016, at 11:39 AM, Jason Altekruse < > > > > > > > > > [email protected] > > > > > > > > > >>> > > > > > > > > > >>> wrote: > > > > > > > > > >>>> > > > > > > > > > >>>> I don't know if there are any specific time > constraints > > > for > > > > > > > getting > > > > > > > > > out > > > > > > > > > >>> the > > > > > > > > > >>>> release, but I'm inclined to go with Vicky on > > DRILL-4477, > > > at > > > > > > least > > > > > > > > > some > > > > > > > > > >>>> investigation into the scope of a fix would be good. I > > > think > > > > > > it's > > > > > > > > > >>>> reasonably big problem whether it's a regression or > not. > > > > > > > > > >>>> > > > > > > > > > >>>> On Mon, Mar 7, 2016 at 11:35 AM, Zelaine Fong < > > > > > > [email protected] > > > > > > > > > > > > > > > > > >>> wrote: > > > > > > > > > >>>> > > > > > > > > > >>>>> Hakim, > > > > > > > > > >>>>> > > > > > > > > > >>>>> Yes, we'll include this in the release. > > > > > > > > > >>>>> > > > > > > > > > >>>>> -- Zelaine > > > > > > > > > >>>>> > > > > > > > > > >>>>> On Mon, Mar 7, 2016 at 9:31 AM, Abdel Hakim Deneche < > > > > > > > > > >>> [email protected] > > > > > > > > > >>>>>> > > > > > > > > > >>>>> wrote: > > > > > > > > > >>>>> > > > > > > > > > >>>>>> If we still have time, I would like to include > > > DRILL-4457 > > > > > [1], > > > > > > > > it's > > > > > > > > > a > > > > > > > > > >>>>> wrong > > > > > > > > > >>>>>> results issue, I already have a fix and it's passing > > all > > > > > > tests, > > > > > > > I > > > > > > > > am > > > > > > > > > >>> just > > > > > > > > > >>>>>> waiting for a review [2] > > > > > > > > > >>>>>> > > > > > > > > > >>>>>> > > > > > > > > > >>>>>> [1] > https://issues.apache.org/jira/browse/DRILL-4457 > > > > > > > > > >>>>>> [2] https://github.com/apache/drill/pull/410 > > > > > > > > > >>>>>> > > > > > > > > > >>>>>> On Mon, Mar 7, 2016 at 4:50 PM, Parth Chandra < > > > > > > > [email protected]> > > > > > > > > > >>> wrote: > > > > > > > > > >>>>>> > > > > > > > > > >>>>>>> Hi guys, > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> I'm still waiting for the following to be > > > reviewed/merged > > > > > by > > > > > > > > today. > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> DRILL-4437 (and others)/pr 394 (Operator unit test > > > > > > framework). > > > > > > > > > >> Waiting > > > > > > > > > >>>>> to > > > > > > > > > >>>>>>> be merged (Jason) > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> DRILL-4372/pr 377(?) (Drill Operators and Functions > > > > should > > > > > > > > > correctly > > > > > > > > > >>>>>> expose > > > > > > > > > >>>>>>> their types within Calcite.) - (Jinfeng to review) > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> DRILL-4313/pr 396 (Improved client randomization. > > > Update > > > > > > JIRA > > > > > > > > with > > > > > > > > > >>>>>>> warnings about using the feature ) > > > (Hanifi/Sudheesh/Paul > > > > - > > > > > > > patch > > > > > > > > > >>>>>> reviewed. > > > > > > > > > >>>>>>> No +1) > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> DRILL-4375/pr 402 (Fix the maven release profile) - > > > > (Jason > > > > > - > > > > > > > > patch > > > > > > > > > >>>>>>> reviewed. Ready to merge?) > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> Thanks > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> Parth > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>> On Sun, Mar 6, 2016 at 12:01 PM, Aditya < > > > > > > > [email protected] > > > > > > > > > > > > > > > > > > >>>>> wrote: > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>>> DRILL-4375/pr 402 - reviewed. > > > > > > > > > >>>>>>>> > > > > > > > > > >>>>>>>> On Sun, Mar 6, 2016 at 12:48 AM, Stefán Baxter < > > > > > > > > > >>>>>>> [email protected]> > > > > > > > > > >>>>>>>> wrote: > > > > > > > > > >>>>>>>> > > > > > > > > > >>>>>>>>> Please review this and then consider as a > potential > > > > > > blocker: > > > > > > > > > >>>>>>>>> > > > > > > > > > >>>>>>>>> https://issues.apache.org/jira/browse/DRILL-4482 > > > > > > > > > >>>>>>>>> > > > > > > > > > >>>>>>>>> > > > > > > > > > >>>>>>>>> > > > > > > > > > >>>>>>>>> On Sat, Mar 5, 2016 at 3:15 AM, Parth Chandra < > > > > > > > > [email protected] > > > > > > > > > > > > > > > > > > > >>>>>>> wrote: > > > > > > > > > >>>>>>>>> > > > > > > > > > >>>>>>>>>> Okay here's the list of JIRA's pending. It > looks > > > > like > > > > > we > > > > > > > > need > > > > > > > > > >>>>> to > > > > > > > > > >>>>>>> get > > > > > > > > > >>>>>>>>> some > > > > > > > > > >>>>>>>>>> more time to get the PRs still under review > > merged, > > > so > > > > > > I'll > > > > > > > > wait > > > > > > > > > >>>>>> over > > > > > > > > > >>>>>>>> the > > > > > > > > > >>>>>>>>>> weekend. > > > > > > > > > >>>>>>>>>> It looks like the PRs that no reviewers assigned > > in > > > > the > > > > > > list > > > > > > > > > >>>>> below > > > > > > > > > >>>>>>> may > > > > > > > > > >>>>>>>>> not > > > > > > > > > >>>>>>>>>> make it into the release. > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> Thanks > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> Committed for 1.6 - > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> DRILL-4384 - Query profile is missing important > > > > > > information > > > > > > > on > > > > > > > > > >>>>>> WebUi > > > > > > > > > >>>>>>> - > > > > > > > > > >>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 support) - Merged. > > > > > > > > > >>>>>>>>>> DRILL-4410/pr 380 (listvector should initiatlize > > > > > bits...) > > > > > > - > > > > > > > > > >>>>> Merged > > > > > > > > > >>>>>>>>>> DRILL-4383/pr 375 (Allow custom configs for S3, > > > > > Kerberos, > > > > > > > > etc) - > > > > > > > > > >>>>>>> Merged > > > > > > > > > >>>>>>>>>> DRILL-4465/pr 401 (Simplify Calcite parsing & > > > planning > > > > > > > > > >>>>>> integration) - > > > > > > > > > >>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>> DRILL-4467 - Baselines need updating. > > > > > > > > > >>>>>>>>>> DRILL-4449/pr 389 (Wrong results when metadata > > cache > > > > is > > > > > > > > used..) > > > > > > > > > >>>>>>> Merged > > > > > > > > > >>>>>>>>>> DRILL-4437 (and others)/pr 394 (Operator unit > test > > > > > > > framework). > > > > > > > > > >>>>>>> Waiting > > > > > > > > > >>>>>>>> to > > > > > > > > > >>>>>>>>>> be merged > > > > > > > > > >>>>>>>>>> DRILL-4449/pr 389 (Wrong results when metadata > > cache > > > > is > > > > > > > > used..) > > > > > > > > > - > > > > > > > > > >>>>>>>>> Waiting > > > > > > > > > >>>>>>>>>> to be merged > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> Need review - > > > > > > > > > >>>>>>>>>> DRILL-4281/pr 400 (Drill should support inbound > > > > > > > impersonation) > > > > > > > > > >>>>>>> (Jacques > > > > > > > > > >>>>>>>>> to > > > > > > > > > >>>>>>>>>> review) > > > > > > > > > >>>>>>>>>> DRILL-4372/pr 377(?) (Drill Operators and > > Functions > > > > > should > > > > > > > > > >>>>>> correctly > > > > > > > > > >>>>>>>>> expose > > > > > > > > > >>>>>>>>>> their types within Calcite.) - (Jinfeng to > review) > > > > > > > > > >>>>>>>>>> DRILL-4313/pr 396 (Improved client > randomization. > > > > > Update > > > > > > > JIRA > > > > > > > > > >>>>> with > > > > > > > > > >>>>>>>>>> warnings about using the feature ) > > > > (Hanifi/Sudheesh/Paul > > > > > > to > > > > > > > > > >>>>>> review.) > > > > > > > > > >>>>>>>>>> DRILL-4375/pr 402 (Fix the maven release > profile) > > - > > > > > > (Aditya > > > > > > > to > > > > > > > > > >>>>>>> review) > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> DRILL-4452/pr 395 (Update Avatica Driver to > latest > > > > > > Calcite) > > > > > > > > > >>>>>>>>>> DRILL-4332/pr 389 (Make vector comparison order > > > stable > > > > > in > > > > > > > test > > > > > > > > > >>>>>>>> framework) > > > > > > > > > >>>>>>>>>> DRILL-4411/pr 381 (hash join over-memory > > condition) > > > > > > > > > >>>>>>>>>> DRILL-4387/pr 379 (GroupScan should not use star > > > > column) > > > > > > > > > >>>>>>>>>> DRILL-4184/pr 372 (support variable length > decimal > > > > > fields > > > > > > in > > > > > > > > > >>>>>> parquet) > > > > > > > > > >>>>>>>>>> DRILL-4120 - dir0 does not work when the > directory > > > > > > structure > > > > > > > > > >>>>>> contains > > > > > > > > > >>>>>>>>> Avro > > > > > > > > > >>>>>>>>>> files - Partial patch available. > > > > > > > > > >>>>>>>>>> DRILL-4203/pr 341 (fix dates written into > parquet > > > > files > > > > > to > > > > > > > > > >>>>> conform > > > > > > > > > >>>>>> to > > > > > > > > > >>>>>>>>>> parquet format spec) > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>> On Fri, Mar 4, 2016 at 6:24 PM, Jacques Nadeau < > > > > > > > > > >>>>> [email protected] > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>>> Awesome. Thanks Chun! > > > > > > > > > >>>>>>>>>>> On Mar 4, 2016 5:51 PM, "Chun Chang" < > > > > > > [email protected]> > > > > > > > > > >>>>>> wrote: > > > > > > > > > >>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>> Jacques submitted a PR for fixing the failed > > > > > baselines. > > > > > > > I've > > > > > > > > > >>>>>>> merged > > > > > > > > > >>>>>>>>>> them > > > > > > > > > >>>>>>>>>>>> into automation master and confirmed the > failed > > > > tests > > > > > > are > > > > > > > > all > > > > > > > > > >>>>>>>> passing > > > > > > > > > >>>>>>>>>>> now. > > > > > > > > > >>>>>>>>>>>> Thanks. > > > > > > > > > >>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>> -Chun > > > > > > > > > >>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>> On Thu, Mar 3, 2016 at 10:48 PM, Jacques > Nadeau > > < > > > > > > > > > >>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>> I think we need to include DRILL-4467 > > > > > > > > > >>>>>>>>>>>>> < > > > https://issues.apache.org/jira/browse/DRILL-4467 > > > > >. > > > > > I > > > > > > > > > >>>>> think > > > > > > > > > >>>>>> it > > > > > > > > > >>>>>>>> is > > > > > > > > > >>>>>>>>> a > > > > > > > > > >>>>>>>>>>> one > > > > > > > > > >>>>>>>>>>>>> line patch and it provides unpredictable > plans > > > at a > > > > > > > minimum > > > > > > > > > >>>>>> but > > > > > > > > > >>>>>>>> may > > > > > > > > > >>>>>>>>>>> also > > > > > > > > > >>>>>>>>>>>>> present invalid result. Still need to think > > > through > > > > > the > > > > > > > > > >>>>>> second > > > > > > > > > >>>>>>>>> half. > > > > > > > > > >>>>>>>>>>> I've > > > > > > > > > >>>>>>>>>>>>> seen this plan instability in some of my > recent > > > > test > > > > > > runs > > > > > > > > > >>>>>> (even > > > > > > > > > >>>>>>>>>> without > > > > > > > > > >>>>>>>>>>>>> Java 8) when running extended HBase tests. > > > > > > > > > >>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>> -- > > > > > > > > > >>>>>>>>>>>>> Jacques Nadeau > > > > > > > > > >>>>>>>>>>>>> CTO and Co-Founder, Dremio > > > > > > > > > >>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>> On Thu, Mar 3, 2016 at 10:02 PM, Parth > Chandra > > < > > > > > > > > > >>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> Updated list (I'll follow up with the folks > > > named > > > > > > here > > > > > > > > > >>>>>>>>>> separately) - > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> Committed for 1.6 - > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4384 - Query profile is missing > > important > > > > > > > > > >>>>> information > > > > > > > > > >>>>>>> on > > > > > > > > > >>>>>>>>>> WebUi > > > > > > > > > >>>>>>>>>>> - > > > > > > > > > >>>>>>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 support) - > Merged. > > > > > > > > > >>>>>>>>>>>>>> DRILL-4410/pr 380 (listvector should > > initiatlize > > > > > > > > > >>>>> bits...) - > > > > > > > > > >>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow custom configs for > > S3, > > > > > > > Kerberos, > > > > > > > > > >>>>>>> etc) > > > > > > > > > >>>>>>>> - > > > > > > > > > >>>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplify Calcite parsing > & > > > > > planning > > > > > > > > > >>>>>>>>>> integration) - > > > > > > > > > >>>>>>>>>>>>>> Waiting to be merged > > > > > > > > > >>>>>>>>>>>>>> DRILL-4437 (and others)/pr 394 (Operator > unit > > > test > > > > > > > > > >>>>>>> framework). > > > > > > > > > >>>>>>>>>>> Waiting > > > > > > > > > >>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>> be merged. > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should support > > inbound > > > > > > > > > >>>>>>> impersonation) > > > > > > > > > >>>>>>>>>>> (Jacques > > > > > > > > > >>>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Drill Operators and > > > > Functions > > > > > > > > > >>>>> should > > > > > > > > > >>>>>>>>>> correctly > > > > > > > > > >>>>>>>>>>>>> expose > > > > > > > > > >>>>>>>>>>>>>> their types within Calcite.) - Waiting for > > Aman > > > to > > > > > > > > > >>>>> review. > > > > > > > > > >>>>>>>>> (Owners: > > > > > > > > > >>>>>>>>>>>>> Hsuan, > > > > > > > > > >>>>>>>>>>>>>> Jinfeng, Aman, Sudheesh) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved client > > > randomization. > > > > > > > Update > > > > > > > > > >>>>>>> JIRA > > > > > > > > > >>>>>>>>> with > > > > > > > > > >>>>>>>>>>>>>> warnings about using the feature ) (Sudheesh > > to > > > > > > review.) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong results when > metadata > > > > cache > > > > > > is > > > > > > > > > >>>>>>> used..) > > > > > > > > > >>>>>>>>>> (Aman > > > > > > > > > >>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4069/pr 352 Enable RPC thread offload > by > > > > > default > > > > > > > > > >>>>>>> (Owner: > > > > > > > > > >>>>>>>>>>>> Sudheesh) > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> Need review - > > > > > > > > > >>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the maven release > > > profile) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4452/pr 395 (Update Avatica Driver to > > > latest > > > > > > > > > >>>>> Calcite) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4332/pr 389 (Make vector comparison > > order > > > > > stable > > > > > > > in > > > > > > > > > >>>>>>> test > > > > > > > > > >>>>>>>>>>>> framework) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4411/pr 381 (hash join over-memory > > > > condition) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan should not use > > star > > > > > > column) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4184/pr 372 (support variable length > > > decimal > > > > > > > fields > > > > > > > > > >>>>>> in > > > > > > > > > >>>>>>>>>> parquet) > > > > > > > > > >>>>>>>>>>>>>> DRILL-4120 - dir0 does not work when the > > > directory > > > > > > > > > >>>>>> structure > > > > > > > > > >>>>>>>>>> contains > > > > > > > > > >>>>>>>>>>>>> Avro > > > > > > > > > >>>>>>>>>>>>>> files - Partial patch available. > > > > > > > > > >>>>>>>>>>>>>> DRILL-4203/pr 341 (fix dates written into > > > parquet > > > > > > files > > > > > > > > > >>>>> to > > > > > > > > > >>>>>>>>> conform > > > > > > > > > >>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>> parquet format spec) > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> Not included (yet) - > > > > > > > > > >>>>>>>>>>>>>> DRILL-3149 - No patch available > > > > > > > > > >>>>>>>>>>>>>> DRILL-4441 - IN operator does not work with > > Avro > > > > > > reader > > > > > > > - > > > > > > > > > >>>>>> No > > > > > > > > > >>>>>>>>> patch > > > > > > > > > >>>>>>>>>>>>>> available > > > > > > > > > >>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive char support - New > > > > feature > > > > > - > > > > > > > > > >>>>> Needs > > > > > > > > > >>>>>>> QA > > > > > > > > > >>>>>>>> - > > > > > > > > > >>>>>>>>>> Not > > > > > > > > > >>>>>>>>>>>>>> included in 1.6 > > > > > > > > > >>>>>>>>>>>>>> DRILL-3623 - Limit 0 should avoid execution > > when > > > > > > > > > >>>>> querying a > > > > > > > > > >>>>>>>> known > > > > > > > > > >>>>>>>>>>>> schema. > > > > > > > > > >>>>>>>>>>>>>> (Need to add limitations of current impl). > > > > Intrusive > > > > > > > > > >>>>>> change; > > > > > > > > > >>>>>>>>> should > > > > > > > > > >>>>>>>>>>> be > > > > > > > > > >>>>>>>>>>>>>> included at beginning of release cycle. > > > > > > > > > >>>>>>>>>>>>>> DRILL-4416/pr 385 (quote path separator) > > (Owner: > > > > > > Hanifi) > > > > > > > > > >>>>> - > > > > > > > > > >>>>>>>> Causes > > > > > > > > > >>>>>>>>>>> leak. > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> Others - > > > > > > > > > >>>>>>>>>>>>>> DRILL-2517 - Already resolved. > > > > > > > > > >>>>>>>>>>>>>> DRILL-3688/pr 382 (skip.header.line.count in > > > > hive). > > > > > - > > > > > > > > > >>>>>> Already > > > > > > > > > >>>>>>>>>> merged. > > > > > > > > > >>>>>>>>>>>> PR > > > > > > > > > >>>>>>>>>>>>>> needs to be closed. > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 9:44 PM, Parth > Chandra > > < > > > > > > > > > >>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>> > > > > > > > > > >>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>> Right. My mistake. Thanks, Jacques, for > > > > reviewing. > > > > > > > > > >>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 9:08 PM, Zelaine > Fong > > < > > > > > > > > > >>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should support > > > inbound > > > > > > > > > >>>>>>>> impersonation) > > > > > > > > > >>>>>>>>>>>>> (Sudheesh > > > > > > > > > >>>>>>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>> Sudheesh is the fixer of DRILL-4281, so I > > > don't > > > > > > think > > > > > > > > > >>>>> he > > > > > > > > > >>>>>>> can > > > > > > > > > >>>>>>>>> be > > > > > > > > > >>>>>>>>>>> the > > > > > > > > > >>>>>>>>>>>>>>>> reviewer :). > > > > > > > > > >>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>> -- Zelaine > > > > > > > > > >>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 6:30 PM, Parth > > Chandra > > > < > > > > > > > > > >>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Here's an updated list with names of > > > reviewers > > > > > > > > > >>>>> added. > > > > > > > > > >>>>>> If > > > > > > > > > >>>>>>>>>> anyone > > > > > > > > > >>>>>>>>>>>> else > > > > > > > > > >>>>>>>>>>>>>> is > > > > > > > > > >>>>>>>>>>>>>>>>> reviewing the open PRs please let me > know. > > > Some > > > > > PRs > > > > > > > > > >>>>>> have > > > > > > > > > >>>>>>>>>> owners > > > > > > > > > >>>>>>>>>>>>> names > > > > > > > > > >>>>>>>>>>>>>>>> that > > > > > > > > > >>>>>>>>>>>>>>>>> I will follow up with. > > > > > > > > > >>>>>>>>>>>>>>>>> Jason, I've included your JIRA in the > list. > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Committed for 1.6 - > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4384 - Query profile is missing > > > important > > > > > > > > > >>>>>>>> information > > > > > > > > > >>>>>>>>> on > > > > > > > > > >>>>>>>>>>>>> WebUi - > > > > > > > > > >>>>>>>>>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 support) - > > > Merged. > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4410/pr 380 (listvector should > > > > initiatlize > > > > > > > > > >>>>>>> bits...) > > > > > > > > > >>>>>>>> - > > > > > > > > > >>>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow custom configs > for > > > S3, > > > > > > > > > >>>>>>> Kerberos, > > > > > > > > > >>>>>>>>>> etc) - > > > > > > > > > >>>>>>>>>>>>>> Merged > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplify Calcite > > parsing & > > > > > > > > > >>>>> planning > > > > > > > > > >>>>>>>>>>>> integration) > > > > > > > > > >>>>>>>>>>>>> - > > > > > > > > > >>>>>>>>>>>>>>>>> Waiting to be merged > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should support > > > inbound > > > > > > > > > >>>>>>>>> impersonation) > > > > > > > > > >>>>>>>>>>>>>>>> (Sudheesh to > > > > > > > > > >>>>>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Drill Operators and > > > > > Functions > > > > > > > > > >>>>>>> should > > > > > > > > > >>>>>>>>>>>> correctly > > > > > > > > > >>>>>>>>>>>>>>>> expose > > > > > > > > > >>>>>>>>>>>>>>>>> their types within Calcite.) - Waiting > for > > > Aman > > > > > to > > > > > > > > > >>>>>>> review. > > > > > > > > > >>>>>>>>>>>> (Owners: > > > > > > > > > >>>>>>>>>>>>>>>> Hsuan, > > > > > > > > > >>>>>>>>>>>>>>>>> Jinfeng, Aman, Sudheesh) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved client > > > > > randomization. > > > > > > > > > >>>>>>> Update > > > > > > > > > >>>>>>>>> JIRA > > > > > > > > > >>>>>>>>>>>> with > > > > > > > > > >>>>>>>>>>>>>>>>> warnings about using the feature ) > > (Sudheesh > > > to > > > > > > > > > >>>>>> review.) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4437 (and others)/pr 394 (Operator > > unit > > > > > test > > > > > > > > > >>>>>>>>> framework). > > > > > > > > > >>>>>>>>>>>>> (Parth > > > > > > > > > >>>>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong results when > > > metadata > > > > > > cache > > > > > > > > > >>>>>> is > > > > > > > > > >>>>>>>>>> used..) > > > > > > > > > >>>>>>>>>>>>> (Aman > > > > > > > > > >>>>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>>>>> review) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4416/pr 385 (quote path separator) > > > > (Owner: > > > > > > > > > >>>>>> Hanifi) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4069/pr 352 Enable RPC thread > offload > > > by > > > > > > > > > >>>>> default > > > > > > > > > >>>>>>>>> (Owner: > > > > > > > > > >>>>>>>>>>>>>> Sudheesh) > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Need review - > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the maven release > > > > profile) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4452/pr 395 (Update Avatica Driver > to > > > > > latest > > > > > > > > > >>>>>>>> Calcite) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4332/pr 389 (Make vector comparison > > > order > > > > > > > > > >>>>> stable > > > > > > > > > >>>>>>> in > > > > > > > > > >>>>>>>>> test > > > > > > > > > >>>>>>>>>>>>>>>> framework) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4411/pr 381 (hash join over-memory > > > > > condition) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan should not > use > > > > star > > > > > > > > > >>>>>> column) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4184/pr 372 (support variable > length > > > > > decimal > > > > > > > > > >>>>>>> fields > > > > > > > > > >>>>>>>> in > > > > > > > > > >>>>>>>>>>>>> parquet) > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4120 - dir0 does not work when the > > > > > directory > > > > > > > > > >>>>>>>> structure > > > > > > > > > >>>>>>>>>>>>> contains > > > > > > > > > >>>>>>>>>>>>>>>> Avro > > > > > > > > > >>>>>>>>>>>>>>>>> files - Partial patch available. > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4203/pr 341 (fix dates written into > > > > parquet > > > > > > > > > >>>>>> files > > > > > > > > > >>>>>>> to > > > > > > > > > >>>>>>>>>>> conform > > > > > > > > > >>>>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>>>>> parquet format spec) > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Not included (yet) - > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3149 - No patch available > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-4441 - IN operator does not work > with > > > > Avro > > > > > > > > > >>>>>> reader > > > > > > > > > >>>>>>> - > > > > > > > > > >>>>>>>> No > > > > > > > > > >>>>>>>>>>> patch > > > > > > > > > >>>>>>>>>>>>>>>>> available > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive char support - > New > > > > > feature > > > > > > > > > >>>>> - > > > > > > > > > >>>>>>>> Needs > > > > > > > > > >>>>>>>>>> QA - > > > > > > > > > >>>>>>>>>>>> Not > > > > > > > > > >>>>>>>>>>>>>>>>> included in 1.6 > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3623 - Limit 0 should avoid > execution > > > > when > > > > > > > > > >>>>>>> querying > > > > > > > > > >>>>>>>> a > > > > > > > > > >>>>>>>>>>> known > > > > > > > > > >>>>>>>>>>>>>>>> schema. > > > > > > > > > >>>>>>>>>>>>>>>>> (Need to add limitations of current > impl). > > > > > > Intrusive > > > > > > > > > >>>>>>>> change; > > > > > > > > > >>>>>>>>>>>> should > > > > > > > > > >>>>>>>>>>>>> be > > > > > > > > > >>>>>>>>>>>>>>>>> included at beginning of release cycle. > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Others - > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-2517 - Already resolved. > > > > > > > > > >>>>>>>>>>>>>>>>> DRILL-3688/pr 382 (skip.header.line.count > > in > > > > > hive). > > > > > > > > > >>>>> - > > > > > > > > > >>>>>>>>> Already > > > > > > > > > >>>>>>>>>>>>> merged. > > > > > > > > > >>>>>>>>>>>>>> PR > > > > > > > > > >>>>>>>>>>>>>>>>> needs to be closed. > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Thanks > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> Parth > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> On Thu, Mar 3, 2016 at 3:21 PM, Jason > > > > Altekruse < > > > > > > > > > >>>>>>>>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>> I have updated the PR for the parquet > date > > > > > > > > > >>>>>> corruption > > > > > > > > > >>>>>>>>> issue > > > > > > > > > >>>>>>>>>>> that > > > > > > > > > >>>>>>>>>>>>>>>> didn't > > > > > > > > > >>>>>>>>>>>>>>>>>> make it into 1.5. > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>> > https://github.com/apache/drill/pull/341 > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > https://issues.apache.org/jira/browse/DRILL-4203 > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>> If this can get reviewed, I think it > would > > > be > > > > > good > > > > > > > > > >>>>>> to > > > > > > > > > >>>>>>>> get > > > > > > > > > >>>>>>>>>> into > > > > > > > > > >>>>>>>>>>>> the > > > > > > > > > >>>>>>>>>>>>>>>>> release. > > > > > > > > > >>>>>>>>>>>>>>>>>> Any takers? > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 11:07 PM, Parth > > > > Chandra < > > > > > > > > > >>>>>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> I've summarized the list of JIRs below. > > > > > > > > > >>>>>>>>>>>>>>>>>>> The first set of pull requests is under > > > > review > > > > > > > > > >>>>> (or > > > > > > > > > >>>>>>>> have > > > > > > > > > >>>>>>>>>> some > > > > > > > > > >>>>>>>>>>>>>>>> reviewer > > > > > > > > > >>>>>>>>>>>>>>>>>>> assigned). > > > > > > > > > >>>>>>>>>>>>>>>>>>> The second set contains pull requests > > that > > > > need > > > > > > > > > >>>>>>>> review. > > > > > > > > > >>>>>>>>> We > > > > > > > > > >>>>>>>>>>>> need > > > > > > > > > >>>>>>>>>>>>>>>>>> committers > > > > > > > > > >>>>>>>>>>>>>>>>>>> to review these. Please volunteer or > > these > > > > will > > > > > > > > > >>>>>> not > > > > > > > > > >>>>>>> be > > > > > > > > > >>>>>>>>>> able > > > > > > > > > >>>>>>>>>>> to > > > > > > > > > >>>>>>>>>>>>>> make > > > > > > > > > >>>>>>>>>>>>>>>> it > > > > > > > > > >>>>>>>>>>>>>>>>>> into > > > > > > > > > >>>>>>>>>>>>>>>>>>> the release. > > > > > > > > > >>>>>>>>>>>>>>>>>>> The third set is Jira's that do not > have > > a > > > > > patch > > > > > > > > > >>>>>>>> and/or > > > > > > > > > >>>>>>>>>>> should > > > > > > > > > >>>>>>>>>>>>> not > > > > > > > > > >>>>>>>>>>>>>>>> be > > > > > > > > > >>>>>>>>>>>>>>>>>>> included because they require deeper > > > > scrutiny. > > > > > > > > > >>>>>>>>>>>>>>>>>>> I'm hoping we can finalize the list of > > PRs > > > > that > > > > > > > > > >>>>>> can > > > > > > > > > >>>>>>> be > > > > > > > > > >>>>>>>>>>>> reviewed > > > > > > > > > >>>>>>>>>>>>> by > > > > > > > > > >>>>>>>>>>>>>>>>> Friday > > > > > > > > > >>>>>>>>>>>>>>>>>>> morning and possibly *finalize the list > > of > > > > > > > > > >>>>> issues > > > > > > > > > >>>>>> to > > > > > > > > > >>>>>>>> be > > > > > > > > > >>>>>>>>>>>> included > > > > > > > > > >>>>>>>>>>>>>> by > > > > > > > > > >>>>>>>>>>>>>>>>>> Friday > > > > > > > > > >>>>>>>>>>>>>>>>>>> end of day* so please take some time to > > > > review > > > > > > > > > >>>>> the > > > > > > > > > >>>>>>>> PRs. > > > > > > > > > >>>>>>>>>>>>>>>>>>> Also note that the QA team has offered > to > > > do > > > > > > > > > >>>>>> sanity > > > > > > > > > >>>>>>>>>> testing > > > > > > > > > >>>>>>>>>>>> once > > > > > > > > > >>>>>>>>>>>>>> we > > > > > > > > > >>>>>>>>>>>>>>>>>> decide > > > > > > > > > >>>>>>>>>>>>>>>>>>> on the final commit to be included, > > before > > > > the > > > > > > > > > >>>>>>> release > > > > > > > > > >>>>>>>>>>>> candidate > > > > > > > > > >>>>>>>>>>>>>> is > > > > > > > > > >>>>>>>>>>>>>>>>>> rolled > > > > > > > > > >>>>>>>>>>>>>>>>>>> out, which helps with the release > > candidate > > > > > > > > > >>>>> moving > > > > > > > > > >>>>>>>>> forward > > > > > > > > > >>>>>>>>>>>>>> smoothly. > > > > > > > > > >>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> Here's the list - > > > > > > > > > >>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Committed for 1.6 -* > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4281/pr 400 (Drill should support > > > > inbound > > > > > > > > > >>>>>>>>>>> impersonation) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4372/pr 377(?) (Drill Operators > and > > > > > > > > > >>>>>> Functions > > > > > > > > > >>>>>>>>> should > > > > > > > > > >>>>>>>>>>>>>> correctly > > > > > > > > > >>>>>>>>>>>>>>>>>> expose > > > > > > > > > >>>>>>>>>>>>>>>>>>> their types within Calcite.) - Waiting > > for > > > > Aman > > > > > > > > > >>>>> to > > > > > > > > > >>>>>>>>> review. > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4313/pr 396 (Improved client > > > > > > > > > >>>>> randomization. > > > > > > > > > >>>>>>>>> Update > > > > > > > > > >>>>>>>>>>> JIRA > > > > > > > > > >>>>>>>>>>>>>> with > > > > > > > > > >>>>>>>>>>>>>>>>>>> warnings about using the feature ) > > Sudheesh > > > > to > > > > > > > > > >>>>>>> review. > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3488/pr 388 (Java 1.8 support) > > Hanifi > > > > to > > > > > > > > > >>>>>>> review > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4437 (and others)/pr 394 > (Operator > > > unit > > > > > > > > > >>>>> test > > > > > > > > > >>>>>>>>>>> framework). > > > > > > > > > >>>>>>>>>>>>>>>> Parth to > > > > > > > > > >>>>>>>>>>>>>>>>>>> review > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4384 - Query profile is missing > > > > important > > > > > > > > > >>>>>>>>>> information > > > > > > > > > >>>>>>>>>>> on > > > > > > > > > >>>>>>>>>>>>>>>> WebUi - > > > > > > > > > >>>>>>>>>>>>>>>>>>> Marked as resolved. Patch not applied? > > > > > > > > > >>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Need review -* > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4465/pr 401 (Simplify Calcite > > > parsing & > > > > > > > > > >>>>>>> planning > > > > > > > > > >>>>>>>>>>>>>> integration) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4375/pr 402 (Fix the maven > release > > > > > > > > > >>>>> profile) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4452/pr 395 (Update Avatica > Driver > > to > > > > > > > > > >>>>> latest > > > > > > > > > >>>>>>>>>> Calcite) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4332/pr 389 (Make vector > comparison > > > > order > > > > > > > > > >>>>>>> stable > > > > > > > > > >>>>>>>>> in > > > > > > > > > >>>>>>>>>>> test > > > > > > > > > >>>>>>>>>>>>>>>>> framework) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4449/pr 389 (Wrong results when > > > > metadata > > > > > > > > > >>>>>> cache > > > > > > > > > >>>>>>>> is > > > > > > > > > >>>>>>>>>>>> used..) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4416/pr 385 (quote path > separator) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4411/pr 381 (hash join > over-memory > > > > > > > > > >>>>>> condition) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4410/pr 380 (listvector should > > > > > initiatlize > > > > > > > > > >>>>>>>>> bits...) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4387/pr 379 (GroupScan should not > > use > > > > > star > > > > > > > > > >>>>>>>> column) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4383/pr 375 (Allow custom configs > > for > > > > S3, > > > > > > > > > >>>>>>>>> Kerberos, > > > > > > > > > >>>>>>>>>>> etc) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4184/pr 372 (support variable > > length > > > > > > > > > >>>>> decimal > > > > > > > > > >>>>>>>>> fields > > > > > > > > > >>>>>>>>>> in > > > > > > > > > >>>>>>>>>>>>>>>> parquet) > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4069/pr 352 Enable RPC thread > > offload > > > > by > > > > > > > > > >>>>>>> default > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4120 - dir0 does not work when > the > > > > > > > > > >>>>> directory > > > > > > > > > >>>>>>>>>> structure > > > > > > > > > >>>>>>>>>>>>>>>> contains > > > > > > > > > >>>>>>>>>>>>>>>>>> Avro > > > > > > > > > >>>>>>>>>>>>>>>>>>> files - Partial patch available. > > > > > > > > > >>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>> *Not included (yet) - * > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3149 - No patch available > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-4441 - IN operator does not work > > with > > > > > Avro > > > > > > > > > >>>>>>>> reader > > > > > > > > > >>>>>>>>> - > > > > > > > > > >>>>>>>>>> No > > > > > > > > > >>>>>>>>>>>>> patch > > > > > > > > > >>>>>>>>>>>>>>>>>>> available > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3745/pr 399 - Hive char support - > > New > > > > > > > > > >>>>>> feature > > > > > > > > > >>>>>>> - > > > > > > > > > >>>>>>>>>> Needs > > > > > > > > > >>>>>>>>>>>> QA - > > > > > > > > > >>>>>>>>>>>>>> Not > > > > > > > > > >>>>>>>>>>>>>>>>>>> included in 1.6 > > > > > > > > > >>>>>>>>>>>>>>>>>>> DRILL-3623 - Limit 0 should avoid > > execution > > > > > when > > > > > > > > > >>>>>>>>> querying > > > > > > > > > >>>>>>>>>> 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 < > > > > > > > > > >>>>>>>>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>>>>>>> wrote: > > > > > > > > > >>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>>> You are welcome, Jacques. > > > > > > > > > >>>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>>> Vick*y *:) > > > > > > > > > >>>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>>> On Wed, Mar 2, 2016 at 3:06 PM, > Jacques > > > > > > > > > >>>>> Nadeau < > > > > > > > > > >>>>>>>>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>>>>>>> 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 < > > > > > > > > > >>>>>>>>>>>>>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>>>> 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 < > > > > > > > > > >>>>>>>>>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>>>>>>>>> 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 < > > > > > > > > > >>>>>>>>>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>>>>>>>>> > > > > > > > > > >>>>>>>>>>>>>>>>>>>> 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 > > > > > > > > > >>>>>> < > > > > > > > > > >>>>>>>>>>>>>>>> [email protected]> > > > > > > > > > >>>>>>>>>>>>>>>>>>> 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 < > > > > > > > > > >>>>>>>>>>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>>>>>>>>>>>>>>>> <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 < > > > > > > > > > >>>>>>>>>>>>>>>>>>>> [email protected] > > > > > > > > > >>>>>>>>>>>>>>>>>>>>>>>>> <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 > > > > > > > > > >>>>>>> > > > > > > > > > >>>>>> > > > > > > > > > >>>>> > > > > > > > > > >>> > > > > > > > > > >>> > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
