[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11944 )

Change subject: IMPALA-7871: Don't load Hive builtins
..

IMPALA-7871: Don't load Hive builtins

Dataload has a step of "Loading Hive builtins" that
loads a bunch of jars into HDFS/S3/etc. Despite
its name, nothing seems to be using these.
Dataload and core tests succeed without this step.

This removes the Hive builtins step and associated
scripts.

Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Reviewed-on: http://gerrit.cloudera.org:8080/11944
Reviewed-by: Tim Armstrong 
Tested-by: Impala Public Jenkins 
---
M testdata/bin/create-load-data.sh
D testdata/bin/load-hive-builtins.sh
M testdata/bin/load-test-warehouse-snapshot.sh
3 files changed, 0 insertions(+), 80 deletions(-)

Approvals:
  Tim Armstrong: Looks good to me, approved
  Impala Public Jenkins: Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Joe McDonnell 
Gerrit-Reviewer: Tim Armstrong 


[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11944 )

Change subject: IMPALA-7871: Don't load Hive builtins
..


Patch Set 2: Verified+1


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 19 Nov 2018 23:27:15 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11944 )

Change subject: IMPALA-7871: Don't load Hive builtins
..


Patch Set 2:

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


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 19 Nov 2018 19:16:00 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11944 )

Change subject: IMPALA-7871: Don't load Hive builtins
..


Patch Set 2:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/1394/ : 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/11944
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 19 Nov 2018 19:06:59 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11944 )

Change subject: IMPALA-7871: Don't load Hive builtins
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 19 Nov 2018 18:55:54 +
Gerrit-HasComments: No


[Impala-ASF-CR] IMPALA-7871: Don't load Hive builtins

2018-11-19 Thread Joe McDonnell (Code Review)
Hello Impala Public Jenkins,

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

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

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

Change subject: IMPALA-7871: Don't load Hive builtins
..

IMPALA-7871: Don't load Hive builtins

Dataload has a step of "Loading Hive builtins" that
loads a bunch of jars into HDFS/S3/etc. Despite
its name, nothing seems to be using these.
Dataload and core tests succeed without this step.

This removes the Hive builtins step and associated
scripts.

Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
---
M testdata/bin/create-load-data.sh
D testdata/bin/load-hive-builtins.sh
M testdata/bin/load-test-warehouse-snapshot.sh
3 files changed, 0 insertions(+), 80 deletions(-)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iaca5ffdaca4b5506e9401b17a7806d37fd7b1844
Gerrit-Change-Number: 11944
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell 
Gerrit-Reviewer: Impala Public Jenkins