[GitHub] [drill] rymarm opened a new pull request, #2662: DRILL-8321: Change kafka_2.13 dependency scope to test

[GitBox]

rymarm opened a new pull request, #2662:
URL: https://github.com/apache/drill/pull/2662

   # [DRILL-8321](https://issues.apache.org/jira/browse/DRILL-8321): Change 
kafka_2.13 dependency scope to test 
   
   ## Description
   Move `org.apache.kafka.kafka_2.13` to test dependency scope. It prevents 
conflict between Scala versions of `org.apache.kafka.kafka_2.13` (Scala 2.13) 
and `com.madhukaraphatak.java-sizeof_2.11` (Scala 2.11). Solves such exceptions 
that could appear during Drill-On-Yarn startup:
   ```
   Caused by: java.util.ServiceConfigurationError: 
com.fasterxml.jackson.databind.Module: Provider 
com.fasterxml.jackson.module.scala.DefaultScalaModule could not be instantiated
   ...
   Caused by: java.lang.NoSuchMethodError: 'scala.collection.immutable.Seq$ 
scala.package$.Seq()'
   ```
   
   Also, it cleans Drill's classpath from unnecessary dependencies.
   
   Probably `org.apache.kafka.kafka_2.13` was added to compile scope by 
mistake. 
   
   ## Documentation
   No changes.
   
   ## Testing
   Unit tests pass successfully. Tried to connect to Kafka and read a topic - 
everything works fine.
   


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

To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org