Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/13432


Change subject: WIP: IMPALA-8586: Support download URLs for CDP
......................................................................

WIP: IMPALA-8586: Support download URLs for CDP

The CDP part of bin/bootstrap_toolchain.py has mostly been added
ad-hoc. One feature that is needed on the CDP side is the ability
to specify the download URL in an IMPALA_*_URL environment
variable.

This adds that support and refactors pretty significant chunks of
bin/bootstrap_toolchain.py. Before I spend more time, I wanted to
see if this direction makes sense.

Currently, there are multiple phases of downloads, each with their
own download functions and peculiarities to account for package
names and destinations for downloads. This changes the logic
so that a package will generate a DownloadUnpackAction that is
completely resolved. It contains everything about what to download
and where to put it as well as a download() function that executes
the appropriate actions. Once there is a list of DownloadUnpackAction
objects, they can all be downloaded and unpacked in a single phase.
This implements different types of packages as subclasses of
DownloadUnpackAction.

Change-Id: I67824fd82b820e68e9f5c87939ec94ca6abadb8c
---
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
2 files changed, 265 insertions(+), 164 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I67824fd82b820e68e9f5c87939ec94ca6abadb8c
Gerrit-Change-Number: 13432
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>

Reply via email to