amrishlal commented on a change in pull request #6716:
URL: https://github.com/apache/incubator-pinot/pull/6716#discussion_r601763718



##########
File path: pom.xml
##########
@@ -806,6 +820,12 @@
           </exclusion>
         </exclusions>
       </dependency>
+      <dependency>
+        <groupId>org.apache.hadoop</groupId>

Review comment:
       Some of the exclusions that we have in pom files may be redundant :-) 
But it either case, there are no conflicting dependencies being pulled in 
because of hadoop-mapreduce-client-core. If a higher-level project includes us 
and runs into conflicts because of our transitive dependencies (say 
dependencies being pulled in by the hadoop libraries that we are pulling in) 
they can always use <exclude>.




-- 
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]



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

Reply via email to