[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 6: Code-Review+2

I think this makes sense, and it mostly impacts Hive 3 configurations, so let's 
get this in.


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Joe McDonnell 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Fri, 26 Apr 2019 23:39:19 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..

Configure Hive 3's HS2 to execute queries using Tez local mode

Hive 3 no longer supports MR execution, so this sets up the appropriate
configuration and classpath so that HS2 can run queries using Tez.

The bulk of this patch is toolchain changes to download Tez itself. The
Tez tarball is slightly odd in that it has no top-level directory, so
the patch changes around bootstrap_toolchain a bit to support creating
its own top-level directory for a component.

The remainder of the patch is some classpath setup and hive-site changes
when Hive 3 is enabled.

So far I tested this manually by setting up a metastore and
impala-config with USE_CDP_HIVE=true, and then connecting to HS2 using

  hive beeline -u 'jdbc:hive2://localhost:11050'

I was able to insert and query data, and was able to verify that queries
like 'select count(*)' were executing via Tez local mode.

NOTE: this patch relies on a custom build of Tez, based on a private
branch. I've submitted a PR to Tez upstream, referenced in the commits
here. Will remove this hack once the PR is accepted and makes its way
into an official build.

Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Reviewed-on: http://gerrit.cloudera.org:8080/12931
Tested-by: Impala Public Jenkins 
Reviewed-by: Joe McDonnell 
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
M fe/pom.xml
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
5 files changed, 90 insertions(+), 10 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Joe McDonnell: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 7
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Joe McDonnell 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 6: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Fri, 26 Apr 2019 11:19:39 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 6:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/2930/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Fri, 26 Apr 2019 07:03:24 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Todd Lipcon (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12931

to look at the new patch set (#6).

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..

Configure Hive 3's HS2 to execute queries using Tez local mode

Hive 3 no longer supports MR execution, so this sets up the appropriate
configuration and classpath so that HS2 can run queries using Tez.

The bulk of this patch is toolchain changes to download Tez itself. The
Tez tarball is slightly odd in that it has no top-level directory, so
the patch changes around bootstrap_toolchain a bit to support creating
its own top-level directory for a component.

The remainder of the patch is some classpath setup and hive-site changes
when Hive 3 is enabled.

So far I tested this manually by setting up a metastore and
impala-config with USE_CDP_HIVE=true, and then connecting to HS2 using

  hive beeline -u 'jdbc:hive2://localhost:11050'

I was able to insert and query data, and was able to verify that queries
like 'select count(*)' were executing via Tez local mode.

NOTE: this patch relies on a custom build of Tez, based on a private
branch. I've submitted a PR to Tez upstream, referenced in the commits
here. Will remove this hack once the PR is accepted and makes its way
into an official build.

Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
M fe/pom.xml
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
5 files changed, 90 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/12931/6
--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-26 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 6:

Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/4080/ 
DRY_RUN=true


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 6
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Fri, 26 Apr 2019 06:10:29 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 5: Verified-1

Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/4070/


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 5
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 25 Apr 2019 23:47:43 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 5:

Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/4070/ 
DRY_RUN=true


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 5
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 25 Apr 2019 22:02:47 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-25 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 5: Code-Review+1


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 5
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 25 Apr 2019 17:34:30 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-25 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 5:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/2904/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 5
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 25 Apr 2019 06:48:28 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-24 Thread Todd Lipcon (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12931

to look at the new patch set (#5).

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..

Configure Hive 3's HS2 to execute queries using Tez local mode

Hive 3 no longer supports MR execution, so this sets up the appropriate
configuration and classpath so that HS2 can run queries using Tez.

The bulk of this patch is toolchain changes to download Tez itself. The
Tez tarball is slightly odd in that it has no top-level directory, so
the patch changes around bootstrap_toolchain a bit to support creating
its own top-level directory for a component.

The remainder of the patch is some classpath setup and hive-site changes
when Hive 3 is enabled.

So far I tested this manually by setting up a metastore and
impala-config with USE_CDP_HIVE=true, and then connecting to HS2 using

  hive beeline -u 'jdbc:hive2://localhost:11050'

I was able to insert and query data, and was able to verify that queries
like 'select count(*)' were executing via Tez local mode.

NOTE: this patch relies on a custom build of Tez, based on a private
branch. I've submitted a PR to Tez upstream, referenced in the commits
here. Will remove this hack once the PR is accepted and makes its way
into an official build.

Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
M fe/pom.xml
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
5 files changed, 90 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/12931/5
--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 5
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 2: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py
File fe/src/test/resources/hive-site.xml.py:

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py@79
PS2, Line 79: # Enable Tez and ACID for Hive 3
> Seems this was set to true by default in cbea5ad3a7500632d2c7586865174d6c9f
Thanks for confirming.



--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 04 Apr 2019 19:15:58 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py
File fe/src/test/resources/hive-site.xml.py:

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py@79
PS2, Line 79: # Enable Tez and ACID for Hive 3
> Yea, I had that set for a while, but then found I was able to remove it and
Seems this was set to true by default in 
cbea5ad3a7500632d2c7586865174d6c9f4499ec TEZ-2333 for Tez 0.7 so no longer 
necessary. Will submit a tez docs patch



--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 04 Apr 2019 18:57:28 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py
File fe/src/test/resources/hive-site.xml.py:

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py@79
PS2, Line 79: # Enable Tez and ACID for Hive 3
> Do we need to have a tez-site.xml as well? Also, according to https://tez.a
Yea, I had that set for a while, but then found I was able to remove it and it 
still worked. It may be that it would be necessary for more complex queries. 
Let me try a multi-stage query over some bigger data and see if I notice either 
a big perf difference or if it's required for correctness or what.



--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 04 Apr 2019 18:34:58 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Vihang Karajgaonkar (Code Review)
Vihang Karajgaonkar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py
File fe/src/test/resources/hive-site.xml.py:

http://gerrit.cloudera.org:8080/#/c/12931/2/fe/src/test/resources/hive-site.xml.py@79
PS2, Line 79: # Enable Tez and ACID for Hive 3
Do we need to have a tez-site.xml as well? Also, according to 
https://tez.apache.org/localmode.html looks like we need 
"tez.runtime.optimize.local.fetch" = "true" although I have no idea on what 
that does.



--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 04 Apr 2019 17:59:38 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12931 )

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..


Patch Set 2:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/2641/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Vihang Karajgaonkar 
Gerrit-Comment-Date: Thu, 04 Apr 2019 16:59:27 +
Gerrit-HasComments: No


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-04 Thread Todd Lipcon (Code Review)
Hello Vihang Karajgaonkar, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/12931

to look at the new patch set (#2).

Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..

Configure Hive 3's HS2 to execute queries using Tez local mode

Hive 3 no longer supports MR execution, so this sets up the appropriate
configuration and classpath so that HS2 can run queries using Tez.

The bulk of this patch is toolchain changes to download Tez itself. The
Tez tarball is slightly odd in that it has no top-level directory, so
the patch changes around bootstrap_toolchain a bit to support creating
its own top-level directory for a component.

The remainder of the patch is some classpath setup and hive-site changes
when Hive 3 is enabled.

So far I tested this manually by setting up a metastore and
impala-config with USE_CDP_HIVE=true, and then connecting to HS2 using

  hive beeline -u 'jdbc:hive2://localhost:11050'

I was able to insert and query data, and was able to verify that queries
like 'select count(*)' were executing via Tez local mode.

NOTE: this patch relies on a custom build of Tez, based on a private
branch. I've submitted a PR to Tez upstream, referenced in the commits
here. Will remove this hack once the PR is accepted and makes its way
into an official build.

Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
4 files changed, 66 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/12931/2
--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Vihang Karajgaonkar 


[Impala-ASF-CR] Configure Hive 3's HS2 to execute queries using Tez local mode

2019-04-03 Thread Todd Lipcon (Code Review)
Hello Vihang Karajgaonkar,

I'd like you to do a code review. Please visit

http://gerrit.cloudera.org:8080/12931

to review the following change.


Change subject: Configure Hive 3's HS2 to execute queries using Tez local mode
..

Configure Hive 3's HS2 to execute queries using Tez local mode

Hive 3 no longer supports MR execution, so this sets up the appropriate
configuration and classpath so that HS2 can run queries using Tez.

The bulk of this patch is toolchain changes to download Tez itself. The
Tez tarball is slightly odd in that it has no top-level directory, so
the patch changes around bootstrap_toolchain a bit to support creating
its own top-level directory for a component.

The remainder of the patch is some classpath setup and hive-site changes
when Hive 3 is enabled.

So far I tested this manually by setting up a metastore and
impala-config with USE_CDP_HIVE=true, and then connecting to HS2 using

  hive beeline -u 'jdbc:hive2://localhost:11050'

I was able to insert and query data, and was able to verify that queries
like 'select count(*)' were executing via Tez local mode.

NOTE: this patch relies on a custom build of Tez, based on a private
branch. I've submitted a PR to Tez upstream, referenced in the commits
here. Will remove this hack once the PR is accepted and makes its way
into an official build.

Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
4 files changed, 65 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/12931/1
--
To view, visit http://gerrit.cloudera.org:8080/12931
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I76e47fbd1d6ff5103d81a8de430d5465dba284cd
Gerrit-Change-Number: 12931
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon 
Gerrit-Reviewer: Vihang Karajgaonkar