Re: [VOTE] Release Apache Drill 1.5.0 RC3

2016-02-12 Thread Abhishek Girish 
+1 (non-binding)

- Built from source, and did basic manual sanity tests.
- Ran Functional Regression tests from Drill Test Framework.

-Abhishek

On Friday, February 12, 2016, Jinfeng Ni  wrote:

> +1 (binding)
>
> * Download src tar ball and ran unit tests and full maven build on Mac.
> * Run yelp tutorial queries against yelp dataset.
> * Run TPC-DS queries through sqline and WebUi.
>
>
>
>
> On Fri, Feb 12, 2016 at 8:46 AM, Aman Sinha  > wrote:
> > +1 (binding)
> > - Downloaded src and built, ran unit tests on my Mac
> > - Manually ran a few queries against TPC-DS
> > - Verified partition pruning, metadata caching was working as expected
> for
> > these test queries
> > - Checked query profile in Web UI
> >
> > looks good !
> > Aman
> >
> > On Thu, Feb 11, 2016 at 9:31 PM, Jacques Nadeau  > wrote:
> >
> >> Download, build, unit tests.
> >> Deploy on small cluster and verify operation of a few distributed
> queries.
> >>
> >> LGTM
> >> +1 (binding)
> >>
> >> --
> >> Jacques Nadeau
> >> CTO and Co-Founder, Dremio
> >>
> >> On Thu, Feb 11, 2016 at 4:18 PM, rahul challapalli <
> >> challapallira...@gmail.com > wrote:
> >>
> >> > +1 (non-binding)
> >> >
> >> > Built from source and ran the Functional and Advanced suites from the
> >> test
> >> > framework.
> >> > Performed some sanity tests against web ui authentication.
> >> >
> >> > On Thu, Feb 11, 2016 at 4:03 PM, Jason Altekruse <
> >> altekruseja...@gmail.com 
> >> > >
> >> > wrote:
> >> >
> >> > > Thanks for the vote Sudheesh!
> >> > >
> >> > > Please, others with time available try out the candidate, the vote
> is
> >> > > supposed to close tomorrow at 7PM Pacific.
> >> > >
> >> > > Thanks,
> >> > > Jason
> >> > >
> >> > > On Thu, Feb 11, 2016 at 3:53 PM, Sudheesh Katkam <
> sudhe...@apache.org >
> >> > > wrote:
> >> > >
> >> > > > +1 (non-binding; committer)
> >> > > >
> >> > > > * downloaded and built from source tar-ball; ran unit tests
> >> > successfully
> >> > > on
> >> > > > Ubuntu
> >> > > > * ran simple queries (including cancellations) in embedded mode on
> >> Mac;
> >> > > > verified states in web UI
> >> > > > * ran simple queries (including cancellations) on a 3 node
> cluster;
> >> > > > verified states in web UI
> >> > > > * verified that queries complete with queuing enabled
> >> > > > * verified md5 and sha1 checksums on binary and src tar-balls, and
> >> > zipped
> >> > > > folder
> >> > > >
> >> > > > Thank you,
> >> > > > Sudheesh
> >> > > >
> >> > > > On Tue, Feb 9, 2016 at 7:38 PM, Jason Altekruse <
> >> > > altekruseja...@gmail.com >
> >> > > > wrote:
> >> > > >
> >> > > > > For anyone who jumped on testing out the release candidate early
> >> I'm
> >> > > > going
> >> > > > > to have to ask you to re-download the artifacts you verified. I
> had
> >> > > > > prepared an earlier version of this candidate (but didn't get a
> >> > chance
> >> > > to
> >> > > > > start the vote) before another regression was identified and
> fixed
> >> > > > today. I
> >> > > > > had forgotten to update the source and binary artifacts on my
> >> apache
> >> > > web
> >> > > > > space with the new ones.
> >> > > > >
> >> > > > > I just uploaded the corrected versions of the artifacts after
> >> > verifying
> >> > > > > their git.properties files to ensure they were the correct
> >> versions.
> >> > > > >
> >> > > > > The last modified dates on the correct versions are between:
> >> > > 10-Feb-2016
> >> > > > > 03:30 and 10-Feb-2016 03:34
> >> > > > >
> >> > > > > (the copy took a few minutes as my home upload speed isn't great
> >> :P)
> >> > > > >
> >> > > > > Sorry about the mistake, everything should be good to go now.
> >> > > > >
> >> > > > > Thanks,
> >> > > > > Jason
> >> > > > >
> >> > > > >
> >> > > > >
> >> > > > > On Tue, Feb 9, 2016 at 7:08 PM, Jason Altekruse <
> >> > > > altekruseja...@gmail.com >
> >> > > > > wrote:
> >> > > > >
> >> > > > > > Hello all,
> >> > > > > >
> >> > > > > > I'd like to propose the forth release candidate (rc3) of
> Apache
> >> > > Drill,
> >> > > > > > version
> >> > > > > > 1.5.0. It covers a total of 60 resolved JIRAs [1]. Thanks to
> >> > everyone
> >> > > > who
> >> > > > > > contributed to this release. This release candidate includes
> >> fixes
> >> > > for
> >> > > > > > DRILL-4235 and DRILL-4380, both regressions found sine the
> last
> >> > > release
> >> > > > > > candidate.
> >> > > > > >
> >> > > > > > I also pulled in two bug fixes (4230, 4349) that had been
> merged
> >> > into
> >> > > > > > master since making the release branch, they looked useful to
> >> > include
> >> > > > and
> >> > > > > > were both had little risk of introducing regressions.
> >> > > > > >
> >> > > > > > The tarball artifacts are hosted at [2] and the maven
> artifacts
> >> are
> >> > > > > hosted
> >> > > > > > at
> >> > > > > > [3]. This release candidate

Re: [VOTE] Release Apache Drill 1.5.0 RC3

2016-02-12 Thread Aman Sinha 
+1 (binding)
- Downloaded src and built, ran unit tests on my Mac
- Manually ran a few queries against TPC-DS
- Verified partition pruning, metadata caching was working as expected for
these test queries
- Checked query profile in Web UI

looks good !
Aman

On Thu, Feb 11, 2016 at 9:31 PM, Jacques Nadeau  wrote:

> Download, build, unit tests.
> Deploy on small cluster and verify operation of a few distributed queries.
>
> LGTM
> +1 (binding)
>
> --
> Jacques Nadeau
> CTO and Co-Founder, Dremio
>
> On Thu, Feb 11, 2016 at 4:18 PM, rahul challapalli <
> challapallira...@gmail.com> wrote:
>
> > +1 (non-binding)
> >
> > Built from source and ran the Functional and Advanced suites from the
> test
> > framework.
> > Performed some sanity tests against web ui authentication.
> >
> > On Thu, Feb 11, 2016 at 4:03 PM, Jason Altekruse <
> altekruseja...@gmail.com
> > >
> > wrote:
> >
> > > Thanks for the vote Sudheesh!
> > >
> > > Please, others with time available try out the candidate, the vote is
> > > supposed to close tomorrow at 7PM Pacific.
> > >
> > > Thanks,
> > > Jason
> > >
> > > On Thu, Feb 11, 2016 at 3:53 PM, Sudheesh Katkam 
> > > wrote:
> > >
> > > > +1 (non-binding; committer)
> > > >
> > > > * downloaded and built from source tar-ball; ran unit tests
> > successfully
> > > on
> > > > Ubuntu
> > > > * ran simple queries (including cancellations) in embedded mode on
> Mac;
> > > > verified states in web UI
> > > > * ran simple queries (including cancellations) on a 3 node cluster;
> > > > verified states in web UI
> > > > * verified that queries complete with queuing enabled
> > > > * verified md5 and sha1 checksums on binary and src tar-balls, and
> > zipped
> > > > folder
> > > >
> > > > Thank you,
> > > > Sudheesh
> > > >
> > > > On Tue, Feb 9, 2016 at 7:38 PM, Jason Altekruse <
> > > altekruseja...@gmail.com>
> > > > wrote:
> > > >
> > > > > For anyone who jumped on testing out the release candidate early
> I'm
> > > > going
> > > > > to have to ask you to re-download the artifacts you verified. I had
> > > > > prepared an earlier version of this candidate (but didn't get a
> > chance
> > > to
> > > > > start the vote) before another regression was identified and fixed
> > > > today. I
> > > > > had forgotten to update the source and binary artifacts on my
> apache
> > > web
> > > > > space with the new ones.
> > > > >
> > > > > I just uploaded the corrected versions of the artifacts after
> > verifying
> > > > > their git.properties files to ensure they were the correct
> versions.
> > > > >
> > > > > The last modified dates on the correct versions are between:
> > > 10-Feb-2016
> > > > > 03:30 and 10-Feb-2016 03:34
> > > > >
> > > > > (the copy took a few minutes as my home upload speed isn't great
> :P)
> > > > >
> > > > > Sorry about the mistake, everything should be good to go now.
> > > > >
> > > > > Thanks,
> > > > > Jason
> > > > >
> > > > >
> > > > >
> > > > > On Tue, Feb 9, 2016 at 7:08 PM, Jason Altekruse <
> > > > altekruseja...@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > Hello all,
> > > > > >
> > > > > > I'd like to propose the forth release candidate (rc3) of Apache
> > > Drill,
> > > > > > version
> > > > > > 1.5.0. It covers a total of 60 resolved JIRAs [1]. Thanks to
> > everyone
> > > > who
> > > > > > contributed to this release. This release candidate includes
> fixes
> > > for
> > > > > > DRILL-4235 and DRILL-4380, both regressions found sine the last
> > > release
> > > > > > candidate.
> > > > > >
> > > > > > I also pulled in two bug fixes (4230, 4349) that had been merged
> > into
> > > > > > master since making the release branch, they looked useful to
> > include
> > > > and
> > > > > > were both had little risk of introducing regressions.
> > > > > >
> > > > > > The tarball artifacts are hosted at [2] and the maven artifacts
> are
> > > > > hosted
> > > > > > at
> > > > > > [3]. This release candidate is based on commit
> > > > > > 3f228d34782741457a14e28b0d1fdbc35a4fd958 located at [4].
> > > > > >
> > > > > > The vote will be open for the next 72 hours ending at 7 PM
> Pacific,
> > > > > > February 12th, 2016.
> > > > > >
> > > > > > [ ] +1
> > > > > > [ ] +0
> > > > > > [ ] -1
> > > > > >
> > > > > > Here's my vote: +1
> > > > > >
> > > > > > Thanks,
> > > > > > Jason
> > > > > >
> > > > > > [1]
> > > > > >
> > > > >
> > > >
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12313820=12332948
> > > > > >
> > > > > > [2] http://people.apache.org/~json/apache-drill-1.5.0.rc3/
> > > > > > [3]
> > > > >
> > https://repository.apache.org/content/repositories/orgapachedrill-1028
> > > > > > [4]
> > > https://github.com/jaltekruse/incubator-drill/tree/drill-1.5.0-rc3
> > > > > >
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] Release Apache Drill 1.5.0 RC3

2016-02-12 Thread Jinfeng Ni 
+1 (binding)

* Download src tar ball and ran unit tests and full maven build on Mac.
* Run yelp tutorial queries against yelp dataset.
* Run TPC-DS queries through sqline and WebUi.




On Fri, Feb 12, 2016 at 8:46 AM, Aman Sinha  wrote:
> +1 (binding)
> - Downloaded src and built, ran unit tests on my Mac
> - Manually ran a few queries against TPC-DS
> - Verified partition pruning, metadata caching was working as expected for
> these test queries
> - Checked query profile in Web UI
>
> looks good !
> Aman
>
> On Thu, Feb 11, 2016 at 9:31 PM, Jacques Nadeau  wrote:
>
>> Download, build, unit tests.
>> Deploy on small cluster and verify operation of a few distributed queries.
>>
>> LGTM
>> +1 (binding)
>>
>> --
>> Jacques Nadeau
>> CTO and Co-Founder, Dremio
>>
>> On Thu, Feb 11, 2016 at 4:18 PM, rahul challapalli <
>> challapallira...@gmail.com> wrote:
>>
>> > +1 (non-binding)
>> >
>> > Built from source and ran the Functional and Advanced suites from the
>> test
>> > framework.
>> > Performed some sanity tests against web ui authentication.
>> >
>> > On Thu, Feb 11, 2016 at 4:03 PM, Jason Altekruse <
>> altekruseja...@gmail.com
>> > >
>> > wrote:
>> >
>> > > Thanks for the vote Sudheesh!
>> > >
>> > > Please, others with time available try out the candidate, the vote is
>> > > supposed to close tomorrow at 7PM Pacific.
>> > >
>> > > Thanks,
>> > > Jason
>> > >
>> > > On Thu, Feb 11, 2016 at 3:53 PM, Sudheesh Katkam 
>> > > wrote:
>> > >
>> > > > +1 (non-binding; committer)
>> > > >
>> > > > * downloaded and built from source tar-ball; ran unit tests
>> > successfully
>> > > on
>> > > > Ubuntu
>> > > > * ran simple queries (including cancellations) in embedded mode on
>> Mac;
>> > > > verified states in web UI
>> > > > * ran simple queries (including cancellations) on a 3 node cluster;
>> > > > verified states in web UI
>> > > > * verified that queries complete with queuing enabled
>> > > > * verified md5 and sha1 checksums on binary and src tar-balls, and
>> > zipped
>> > > > folder
>> > > >
>> > > > Thank you,
>> > > > Sudheesh
>> > > >
>> > > > On Tue, Feb 9, 2016 at 7:38 PM, Jason Altekruse <
>> > > altekruseja...@gmail.com>
>> > > > wrote:
>> > > >
>> > > > > For anyone who jumped on testing out the release candidate early
>> I'm
>> > > > going
>> > > > > to have to ask you to re-download the artifacts you verified. I had
>> > > > > prepared an earlier version of this candidate (but didn't get a
>> > chance
>> > > to
>> > > > > start the vote) before another regression was identified and fixed
>> > > > today. I
>> > > > > had forgotten to update the source and binary artifacts on my
>> apache
>> > > web
>> > > > > space with the new ones.
>> > > > >
>> > > > > I just uploaded the corrected versions of the artifacts after
>> > verifying
>> > > > > their git.properties files to ensure they were the correct
>> versions.
>> > > > >
>> > > > > The last modified dates on the correct versions are between:
>> > > 10-Feb-2016
>> > > > > 03:30 and 10-Feb-2016 03:34
>> > > > >
>> > > > > (the copy took a few minutes as my home upload speed isn't great
>> :P)
>> > > > >
>> > > > > Sorry about the mistake, everything should be good to go now.
>> > > > >
>> > > > > Thanks,
>> > > > > Jason
>> > > > >
>> > > > >
>> > > > >
>> > > > > On Tue, Feb 9, 2016 at 7:08 PM, Jason Altekruse <
>> > > > altekruseja...@gmail.com>
>> > > > > wrote:
>> > > > >
>> > > > > > Hello all,
>> > > > > >
>> > > > > > I'd like to propose the forth release candidate (rc3) of Apache
>> > > Drill,
>> > > > > > version
>> > > > > > 1.5.0. It covers a total of 60 resolved JIRAs [1]. Thanks to
>> > everyone
>> > > > who
>> > > > > > contributed to this release. This release candidate includes
>> fixes
>> > > for
>> > > > > > DRILL-4235 and DRILL-4380, both regressions found sine the last
>> > > release
>> > > > > > candidate.
>> > > > > >
>> > > > > > I also pulled in two bug fixes (4230, 4349) that had been merged
>> > into
>> > > > > > master since making the release branch, they looked useful to
>> > include
>> > > > and
>> > > > > > were both had little risk of introducing regressions.
>> > > > > >
>> > > > > > The tarball artifacts are hosted at [2] and the maven artifacts
>> are
>> > > > > hosted
>> > > > > > at
>> > > > > > [3]. This release candidate is based on commit
>> > > > > > 3f228d34782741457a14e28b0d1fdbc35a4fd958 located at [4].
>> > > > > >
>> > > > > > The vote will be open for the next 72 hours ending at 7 PM
>> Pacific,
>> > > > > > February 12th, 2016.
>> > > > > >
>> > > > > > [ ] +1
>> > > > > > [ ] +0
>> > > > > > [ ] -1
>> > > > > >
>> > > > > > Here's my vote: +1
>> > > > > >
>> > > > > > Thanks,
>> > > > > > Jason
>> > > > > >
>> > > > > > [1]
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12313820=12332948
>> > > > > >
>> > > > > > [2] http://people.apache.org/~json/apache-drill-1.5.0.rc3/

Accumulo integration

2016-02-12 Thread Rodgers, Paul 
Are there any plans to integrate with Accumulo? This would be very useful for 
me.

I’m curious how much difficulty this would involve, and how this would fit in 
with Accumulo’s cell-level visibility pattern.

Paul



The information contained in this e-mail is confidential and/or proprietary to 
Capital One and/or its affiliates and may only be used solely in performance of 
work or services for Capital One. The information transmitted herewith is 
intended only for use by the individual or entity to which it is addressed. If 
the reader of this message is not the intended recipient, you are hereby 
notified that any review, retransmission, dissemination, distribution, copying 
or other use of, or taking of any action in reliance upon this information is 
strictly prohibited. If you have received this communication in error, please 
contact the sender and delete the material from your computer.

[GitHub] drill pull request: DRILL-3623: Optimize limit 0 queries

2016-02-12 Thread sudheeshkatkam 
Github user sudheeshkatkam commented on the pull request:

https://github.com/apache/drill/pull/364#issuecomment-183425694
  
The update does not handle all type (because some types were out of scope). 
The [latest patch](https://github.com/sudheeshkatkam/drill/commits/limit_zero) 
(WIP) is rebased on top of Hsuan's work.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

RE: [VOTE] Release Apache Drill 1.5.0 RC3

2016-02-12 Thread Norris Lee 
+1 (non-binding)

Built from source on CentOS. Ran queries against JSON, CSV, TSV, parquet, Hive 
tables through the ODBC driver.

Norris

-Original Message-
From: Abhishek Girish [mailto:abhishek.gir...@gmail.com] 
Sent: Friday, February 12, 2016 9:12 AM
To: dev@drill.apache.org
Subject: Re: [VOTE] Release Apache Drill 1.5.0 RC3

+1 (non-binding)

- Built from source, and did basic manual sanity tests.
- Ran Functional Regression tests from Drill Test Framework.

-Abhishek

On Friday, February 12, 2016, Jinfeng Ni  wrote:

> +1 (binding)
>
> * Download src tar ball and ran unit tests and full maven build on Mac.
> * Run yelp tutorial queries against yelp dataset.
> * Run TPC-DS queries through sqline and WebUi.
>
>
>
>
> On Fri, Feb 12, 2016 at 8:46 AM, Aman Sinha  > wrote:
> > +1 (binding)
> > - Downloaded src and built, ran unit tests on my Mac
> > - Manually ran a few queries against TPC-DS
> > - Verified partition pruning, metadata caching was working as 
> > expected
> for
> > these test queries
> > - Checked query profile in Web UI
> >
> > looks good !
> > Aman
> >
> > On Thu, Feb 11, 2016 at 9:31 PM, Jacques Nadeau  > wrote:
> >
> >> Download, build, unit tests.
> >> Deploy on small cluster and verify operation of a few distributed
> queries.
> >>
> >> LGTM
> >> +1 (binding)
> >>
> >> --
> >> Jacques Nadeau
> >> CTO and Co-Founder, Dremio
> >>
> >> On Thu, Feb 11, 2016 at 4:18 PM, rahul challapalli < 
> >> challapallira...@gmail.com > wrote:
> >>
> >> > +1 (non-binding)
> >> >
> >> > Built from source and ran the Functional and Advanced suites from 
> >> > the
> >> test
> >> > framework.
> >> > Performed some sanity tests against web ui authentication.
> >> >
> >> > On Thu, Feb 11, 2016 at 4:03 PM, Jason Altekruse <
> >> altekruseja...@gmail.com 
> >> > >
> >> > wrote:
> >> >
> >> > > Thanks for the vote Sudheesh!
> >> > >
> >> > > Please, others with time available try out the candidate, the 
> >> > > vote
> is
> >> > > supposed to close tomorrow at 7PM Pacific.
> >> > >
> >> > > Thanks,
> >> > > Jason
> >> > >
> >> > > On Thu, Feb 11, 2016 at 3:53 PM, Sudheesh Katkam <
> sudhe...@apache.org >
> >> > > wrote:
> >> > >
> >> > > > +1 (non-binding; committer)
> >> > > >
> >> > > > * downloaded and built from source tar-ball; ran unit tests
> >> > successfully
> >> > > on
> >> > > > Ubuntu
> >> > > > * ran simple queries (including cancellations) in embedded 
> >> > > > mode on
> >> Mac;
> >> > > > verified states in web UI
> >> > > > * ran simple queries (including cancellations) on a 3 node
> cluster;
> >> > > > verified states in web UI
> >> > > > * verified that queries complete with queuing enabled
> >> > > > * verified md5 and sha1 checksums on binary and src 
> >> > > > tar-balls, and
> >> > zipped
> >> > > > folder
> >> > > >
> >> > > > Thank you,
> >> > > > Sudheesh
> >> > > >
> >> > > > On Tue, Feb 9, 2016 at 7:38 PM, Jason Altekruse <
> >> > > altekruseja...@gmail.com >
> >> > > > wrote:
> >> > > >
> >> > > > > For anyone who jumped on testing out the release candidate 
> >> > > > > early
> >> I'm
> >> > > > going
> >> > > > > to have to ask you to re-download the artifacts you 
> >> > > > > verified. I
> had
> >> > > > > prepared an earlier version of this candidate (but didn't 
> >> > > > > get a
> >> > chance
> >> > > to
> >> > > > > start the vote) before another regression was identified 
> >> > > > > and
> fixed
> >> > > > today. I
> >> > > > > had forgotten to update the source and binary artifacts on 
> >> > > > > my
> >> apache
> >> > > web
> >> > > > > space with the new ones.
> >> > > > >
> >> > > > > I just uploaded the corrected versions of the artifacts 
> >> > > > > after
> >> > verifying
> >> > > > > their git.properties files to ensure they were the correct
> >> versions.
> >> > > > >
> >> > > > > The last modified dates on the correct versions are between:
> >> > > 10-Feb-2016
> >> > > > > 03:30 and 10-Feb-2016 03:34
> >> > > > >
> >> > > > > (the copy took a few minutes as my home upload speed isn't 
> >> > > > > great
> >> :P)
> >> > > > >
> >> > > > > Sorry about the mistake, everything should be good to go now.
> >> > > > >
> >> > > > > Thanks,
> >> > > > > Jason
> >> > > > >
> >> > > > >
> >> > > > >
> >> > > > > On Tue, Feb 9, 2016 at 7:08 PM, Jason Altekruse <
> >> > > > altekruseja...@gmail.com >
> >> > > > > wrote:
> >> > > > >
> >> > > > > > Hello all,
> >> > > > > >
> >> > > > > > I'd like to propose the forth release candidate (rc3) of
> Apache
> >> > > Drill,
> >> > > > > > version
> >> > > > > > 1.5.0. It covers a total of 60 resolved JIRAs [1]. Thanks 
> >> > > > > > to
> >> > everyone
> >> > > > who
> >> > > > > > contributed to this release. This release candidate 
> >> > > > > > includes
> >> fixes
> >> > > for
> >> > > > > > DRILL-4235 and DRILL-4380, both regressions found sine 
>

Re: Query profile is broken on latest master branch?

2016-02-12 Thread Jinfeng Ni 
After doing a binary search over recent commits, seems the patch of
"DRILL-3581: Upgrade HPPC to 0.7.1" is the cause of broken query
profiles [1].  The query profile on the commits before DRILL-3581
looks fine.


[1] 
https://github.com/apache/drill/commit/d27127c94d5c08306697a5627a1bac5f144abb22


On Fri, Feb 12, 2016 at 7:47 PM, Jinfeng Ni  wrote:
> I built drill from master branch and started in embedded mode. Then,
> run a query and checked the query profile through WebUI. However,
> seems that the fragment profiles , operator profiles and visualized
> plan are all empty. I tried both Mac and CentOS. Seems the same
> problem.
>
> Has anyone experience same problem? or maybe some setup is messed up
> on my machine?
>
> Thanks!

[RESULT][VOTE] Release Apache Drill 1.5.0 RC3

2016-02-12 Thread Jason Altekruse 
*The vote* passes. Thanks everyone for your time *voting*. Final Tally:

4 x +1 (binding)
Jason
Jacques
Aman
Jinfeng


4 x +1 (non-binding)
Sudheesh
Rahul
Abhishek
Norris

No -1s.

I'll push the *release* artifacts and send an announcement once propagated.

Thanks,
Jason

On Fri, Feb 12, 2016 at 10:31 AM, Norris Lee  wrote:

> +1 (non-binding)
>
> Built from source on CentOS. Ran queries against JSON, CSV, TSV, parquet,
> Hive tables through the ODBC driver.
>
> Norris
>
> -Original Message-
> From: Abhishek Girish [mailto:abhishek.gir...@gmail.com]
> Sent: Friday, February 12, 2016 9:12 AM
> To: dev@drill.apache.org
> Subject: Re: [VOTE] Release Apache Drill 1.5.0 RC3
>
> +1 (non-binding)
>
> - Built from source, and did basic manual sanity tests.
> - Ran Functional Regression tests from Drill Test Framework.
>
> -Abhishek
>
> On Friday, February 12, 2016, Jinfeng Ni  wrote:
>
> > +1 (binding)
> >
> > * Download src tar ball and ran unit tests and full maven build on Mac.
> > * Run yelp tutorial queries against yelp dataset.
> > * Run TPC-DS queries through sqline and WebUi.
> >
> >
> >
> >
> > On Fri, Feb 12, 2016 at 8:46 AM, Aman Sinha  > > wrote:
> > > +1 (binding)
> > > - Downloaded src and built, ran unit tests on my Mac
> > > - Manually ran a few queries against TPC-DS
> > > - Verified partition pruning, metadata caching was working as
> > > expected
> > for
> > > these test queries
> > > - Checked query profile in Web UI
> > >
> > > looks good !
> > > Aman
> > >
> > > On Thu, Feb 11, 2016 at 9:31 PM, Jacques Nadeau  > > wrote:
> > >
> > >> Download, build, unit tests.
> > >> Deploy on small cluster and verify operation of a few distributed
> > queries.
> > >>
> > >> LGTM
> > >> +1 (binding)
> > >>
> > >> --
> > >> Jacques Nadeau
> > >> CTO and Co-Founder, Dremio
> > >>
> > >> On Thu, Feb 11, 2016 at 4:18 PM, rahul challapalli <
> > >> challapallira...@gmail.com > wrote:
> > >>
> > >> > +1 (non-binding)
> > >> >
> > >> > Built from source and ran the Functional and Advanced suites from
> > >> > the
> > >> test
> > >> > framework.
> > >> > Performed some sanity tests against web ui authentication.
> > >> >
> > >> > On Thu, Feb 11, 2016 at 4:03 PM, Jason Altekruse <
> > >> altekruseja...@gmail.com 
> > >> > >
> > >> > wrote:
> > >> >
> > >> > > Thanks for the vote Sudheesh!
> > >> > >
> > >> > > Please, others with time available try out the candidate, the
> > >> > > vote
> > is
> > >> > > supposed to close tomorrow at 7PM Pacific.
> > >> > >
> > >> > > Thanks,
> > >> > > Jason
> > >> > >
> > >> > > On Thu, Feb 11, 2016 at 3:53 PM, Sudheesh Katkam <
> > sudhe...@apache.org >
> > >> > > wrote:
> > >> > >
> > >> > > > +1 (non-binding; committer)
> > >> > > >
> > >> > > > * downloaded and built from source tar-ball; ran unit tests
> > >> > successfully
> > >> > > on
> > >> > > > Ubuntu
> > >> > > > * ran simple queries (including cancellations) in embedded
> > >> > > > mode on
> > >> Mac;
> > >> > > > verified states in web UI
> > >> > > > * ran simple queries (including cancellations) on a 3 node
> > cluster;
> > >> > > > verified states in web UI
> > >> > > > * verified that queries complete with queuing enabled
> > >> > > > * verified md5 and sha1 checksums on binary and src
> > >> > > > tar-balls, and
> > >> > zipped
> > >> > > > folder
> > >> > > >
> > >> > > > Thank you,
> > >> > > > Sudheesh
> > >> > > >
> > >> > > > On Tue, Feb 9, 2016 at 7:38 PM, Jason Altekruse <
> > >> > > altekruseja...@gmail.com >
> > >> > > > wrote:
> > >> > > >
> > >> > > > > For anyone who jumped on testing out the release candidate
> > >> > > > > early
> > >> I'm
> > >> > > > going
> > >> > > > > to have to ask you to re-download the artifacts you
> > >> > > > > verified. I
> > had
> > >> > > > > prepared an earlier version of this candidate (but didn't
> > >> > > > > get a
> > >> > chance
> > >> > > to
> > >> > > > > start the vote) before another regression was identified
> > >> > > > > and
> > fixed
> > >> > > > today. I
> > >> > > > > had forgotten to update the source and binary artifacts on
> > >> > > > > my
> > >> apache
> > >> > > web
> > >> > > > > space with the new ones.
> > >> > > > >
> > >> > > > > I just uploaded the corrected versions of the artifacts
> > >> > > > > after
> > >> > verifying
> > >> > > > > their git.properties files to ensure they were the correct
> > >> versions.
> > >> > > > >
> > >> > > > > The last modified dates on the correct versions are between:
> > >> > > 10-Feb-2016
> > >> > > > > 03:30 and 10-Feb-2016 03:34
> > >> > > > >
> > >> > > > > (the copy took a few minutes as my home upload speed isn't
> > >> > > > > great
> > >> :P)
> > >> > > > >
> > >> > > > > Sorry about the mistake, everything should be good to go now.
> > >> > > > >
> > >> > > > > Thanks,
> > >> > > > > Jason
> > >> > > > >
> > >> > > > >
>

[GitHub] drill pull request: DRILL-4275: create TransientStore for short-li...

2016-02-12 Thread sudheeshkatkam 
Github user sudheeshkatkam commented on a diff in the pull request:

https://github.com/apache/drill/pull/374#discussion_r52797746
  
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -120,7 +120,7 @@ drill.exec: {
 affinity.factor: 1.2
   },
   sys.store.provider: {
-class: "org.apache.drill.exec.store.sys.zk.ZkPStoreProvider",
+class: 
"org.apache.drill.exec.store.sys.store.provider.ZookeeperPersistentStoreProvider",
--- End diff --

Upgrade and downgrade process can be affected by this change (e.g. when 
overridden conf. file has "..._ZkPStoreProvider_"), right?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request: DRILL-4275: create TransientStore for short-li...

2016-02-12 Thread hnfgns 
Github user hnfgns commented on a diff in the pull request:

https://github.com/apache/drill/pull/374#discussion_r52817125
  
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -120,7 +120,7 @@ drill.exec: {
 affinity.factor: 1.2
   },
   sys.store.provider: {
-class: "org.apache.drill.exec.store.sys.zk.ZkPStoreProvider",
+class: 
"org.apache.drill.exec.store.sys.store.provider.ZookeeperPersistentStoreProvider",
--- End diff --

Sure. Overriding config with what is already a default 
value(ZkPStoreProvider) seems odd though. Back to your point, this patch 
proposes a major design/API change around old E/PStore so all references to 
ZkPStoreProvider will fail following this patch.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request: DRILL-4275: create TransientStore for short-li...

2016-02-12 Thread sudheeshkatkam 
Github user sudheeshkatkam commented on a diff in the pull request:

https://github.com/apache/drill/pull/374#discussion_r52818257
  
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -120,7 +120,7 @@ drill.exec: {
 affinity.factor: 1.2
   },
   sys.store.provider: {
-class: "org.apache.drill.exec.store.sys.zk.ZkPStoreProvider",
+class: 
"org.apache.drill.exec.store.sys.store.provider.ZookeeperPersistentStoreProvider",
--- End diff --

We do that on the perf. cluster :P

In fact, any other _\*PstoreProvider_ has to change 
_\*PersistentStoreProvider_ after this patch (and reverse if downgrading).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request: DRILL-4382: Remove dependency on drill-logical...

2016-02-12 Thread jacques-n 
Github user jacques-n commented on the pull request:

https://github.com/apache/drill/pull/373#issuecomment-183581100
  
Changes look good to me.  +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Query profile is broken on latest master branch?

2016-02-12 Thread Jinfeng Ni 
I built drill from master branch and started in embedded mode. Then,
run a query and checked the query profile through WebUI. However,
seems that the fragment profiles , operator profiles and visualized
plan are all empty. I tried both Mac and CentOS. Seems the same
problem.

Has anyone experience same problem? or maybe some setup is messed up
on my machine?

Thanks!