Tim Armstrong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/13279 )

Change subject: IMPALA-8527: prune maven repo dependencies
......................................................................

IMPALA-8527: prune maven repo dependencies

We transitively pull in references to repository.apache.org,
which in turn means we'll go looking for most of our dependencies
there. Downloading from repository.apache.org occasionally hangs,
so there's a high probability of a build getting stuck.

I was able to disable repository.apache.org entirely - the same packages
are available from other repositories that we don't see the same
issues with.

Locating snapshot versions is very chatty - we reach out to
mvnrepository.com and apache.org repeatedly, but I don't think
we actually need to consume any snapshots from them. So I
tried to minimise the number of repositories that we'll consume
snapshots from - I think we only intend to download snapshots
from the CDH repo.

Also remove the plugin snapshots repository. We historically
needed it because we used a snapshot version of the cup plugin,
but that was fixed by IMPALA-3418. Otherwise depending on plugin
snapshots seems like a bad idea.

Change-Id: I08e1f1b7d7742edd61179ee52b5e268c3b4dc61d
Reviewed-on: http://gerrit.cloudera.org:8080/13279
Reviewed-by: Fredy Wijaya <[email protected]>
Reviewed-by: Todd Lipcon <[email protected]>
Tested-by: Tim Armstrong <[email protected]>
---
M impala-parent/pom.xml
1 file changed, 38 insertions(+), 9 deletions(-)

Approvals:
  Fredy Wijaya: Looks good to me, but someone else must approve
  Todd Lipcon: Looks good to me, approved
  Tim Armstrong: Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I08e1f1b7d7742edd61179ee52b5e268c3b4dc61d
Gerrit-Change-Number: 13279
Gerrit-PatchSet: 7
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to