[SPARK-18835] Do not expose shaded types in JavaTypeInference API

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1.0
• Fix Version/s: 2.1.0
• Component/s: SQL
• Assignee: Marcelo Masiero Vanzin
• Reporter: Marcelo Masiero Vanzin
• Created: 12/Dec/16 23:53
• Updated: 15/Dec/16 05:05
• Resolved: 13/Dec/16 18:02

Description

Currently, inferDataType(TypeToken) is called from a different maven module, and because we shade Guava, that sometimes leads to errors (e.g. when running tests using maven):

udf3Test(test.org.apache.spark.sql.JavaUDFSuite)  Time elapsed: 0.084 sec  <<< ERROR!
java.lang.NoSuchMethodError: org.apache.spark.sql.catalyst.JavaTypeInference$.inferDataType(Lcom/google/common/reflect/TypeToken;)Lscala/Tuple2;
        at test.org.apache.spark.sql.JavaUDFSuite.udf3Test(JavaUDFSuite.java:107)


Results :

Tests in error: 
  JavaUDFSuite.udf3Test:107 » NoSuchMethod org.apache.spark.sql.catalyst.JavaTyp...

Instead, we shouldn't expose Guava types in these APIs.

Issue Links

• links to: [Github] Pull Request #16260 (vanzin)