[ 
https://issues.apache.org/jira/browse/HADOOP-17288?focusedWorklogId=502527&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-502527
 ]

ASF GitHub Bot logged work on HADOOP-17288:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 20/Oct/20 06:44
            Start Date: 20/Oct/20 06:44
    Worklog Time Spent: 10m 
      Work Description: bolerio commented on a change in pull request #2342:
URL: https://github.com/apache/hadoop/pull/2342#discussion_r508246398



##########
File path: hadoop-common-project/hadoop-auth/pom.xml
##########
@@ -234,6 +235,24 @@
           
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml</excludeFilterFile>
         </configuration>
       </plugin>
+      <plugin>
+        <groupId>com.google.code.maven-replacer-plugin</groupId>
+        <artifactId>replacer</artifactId>
+        <executions>
+          <execution>
+            <id>replace-sources</id>
+            <configuration>
+              <skip>false</skip>
+            </configuration>
+          </execution>
+          <execution>
+            <id>replace-test-sources</id>
+            <configuration>
+              <skip>false</skip>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>

Review comment:
       @ayushtkn et al.
   
   I recently did similar work as this PR on top of the 3.3.0 release, using 
the exact same approach with the replacer plugin and renaming the guava 
namespace to the Hadoop thirdparty shaded version throughout the whole 
codebase. The short term goal was to solve the Hive incompatibility issue with 
Spark. The change worked, the Hive unit tests within the Spark codebase largely 
pass, no more linkage error.  However, a guava dependency still needed to be 
declared in the Hadoop pom.xml to overcome an enforcerer error about 
conflicting Guava versions that show up as transitive dependencies.  
   
   Hope this is helpful information. If there is interest in the community, I 
make a PR or otherwise make that available for testing?




----------------------------------------------------------------
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:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 502527)
    Time Spent: 3h 20m  (was: 3h 10m)

> Use shaded guava from thirdparty
> --------------------------------
>
>                 Key: HADOOP-17288
>                 URL: https://issues.apache.org/jira/browse/HADOOP-17288
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Ayush Saxena
>            Assignee: Ayush Saxena
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.4.0
>
>          Time Spent: 3h 20m
>  Remaining Estimate: 0h
>
> Use the shaded version of guava in hadoop-thirdparty



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to