Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16342 )
Change subject: IMPALA-10073: Create shaded dependency for S3A and aws-java-sdk-bundle ...................................................................... IMPALA-10073: Create shaded dependency for S3A and aws-java-sdk-bundle The aws-java-sdk-bundle is one of the largest dependencies in the Impala Docker images and continues to grow. The jar includes SDKs for every single AWS service. This patch removes most of the unnecessary SDKs from the aws-java-sdk-bundle, thus drastically decreasing the size of the dependency. The Maven shade plugin is used to do this, and the implementation is similar to what is currently done for the hive-exec jar. This patch takes a conservative approach to removing packages from the aws-java-sdk-bundle jar, and I ensured no direct dependencies of the S3 SDK were removed. The idea is to only remove dependencies that S3A would never conceivably need. Given the huge number of AWS services, I only focused on removing the largest SDKs (the size of each SDK is estimated by the number of classes in the SDK). This decreases the size of the Docker images by about 100 MB. Testing: * Ran core tests against S3 Change-Id: I0939f73be986f83cc1fd07921563b4d9201780f2 Reviewed-on: http://gerrit.cloudera.org:8080/16342 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M CMakeLists.txt M ext-data-source/CMakeLists.txt M fe/CMakeLists.txt M fe/pom.xml D shaded-deps/.gitignore C shaded-deps/hive-exec/CMakeLists.txt R shaded-deps/hive-exec/pom.xml R shaded-deps/s3a-aws-sdk/CMakeLists.txt A shaded-deps/s3a-aws-sdk/pom.xml 9 files changed, 212 insertions(+), 9 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16342 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0939f73be986f83cc1fd07921563b4d9201780f2 Gerrit-Change-Number: 16342 Gerrit-PatchSet: 3 Gerrit-Owner: Sahil Takiar <stak...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
