Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
In core/test, it turns out that selenium is bringing in guava 15,
but the rest of Spark we use guava 14, which is evicted. Since
we made catalyst/test depend on core/test, and hive/test depends
on catalyst/test, hive tests are getting guava 15, which results
in java.lang.IllegalAccessError because of a dependency conflict.

This is resolved by excluding guava 15 from selenium in core/test.
  • Loading branch information
Andrew Or committed May 28, 2015
1 parent ee22cda commit 8882581
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,12 @@
<dependency>
<groupId>org.seleniumhq.selenium</groupId>
<artifactId>selenium-java</artifactId>
<exclusions>
<exclusion>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
</exclusion>
</exclusions>
<scope>test</scope>
</dependency>
<!-- Added for selenium: -->
Expand Down

0 comments on commit 8882581

Please sign in to comment.