bhavikpatel9977 commented on a change in pull request #3411: ZEPPELIN-4272:
Zeppelin fails to compile when hadoop3 is enabled
URL: https://github.com/apache/zeppelin/pull/3411#discussion_r318885773
##########
File path: zeppelin-zengine/pom.xml
##########
@@ -312,6 +308,12 @@
</activation>
<dependencies>
+ <dependency>
+ <groupId>com.amazonaws</groupId>
Review comment:
> Any specific reason this is moved from there to only `hadoop2` profile?
Hope this does not cause any problem with `hadoop3` profile.
Initially this dependency is common for hadoop2 & hadoop3 but for hadoop3
we have added `hadoop-aws` dependency which clone all the required aws, s3
related classes and there are some conflict between the multiple versions of
jar so we have removed `aws-java-sdk-s3` but forgot to added it in hadoop2 and
that was the main reason of failing CI builds.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
