From f11f1c738771339e4031c313f759fa555524f3b3 Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 21 Jan 2016 08:33:22 -0700 Subject: [PATCH 1/3] improved error message --- .../scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala index b5de60cdb7b76..3d34b15d6784a 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala @@ -406,7 +406,7 @@ object JavaTypeInference { expressions.Literal(fieldName) :: extractorFor(fieldValue, fieldType) :: Nil }) } else { - throw new UnsupportedOperationException(s"no encoder found for ${other.getName}") + throw new UnsupportedOperationException(s"Cannot infer type for Java class ${other.getName} because it is not bean-compliant}") } } } From 5600a07d74117fbdad3d058d7e798418ae5ef884 Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 21 Jan 2016 08:41:18 -0700 Subject: [PATCH 2/3] fix typo --- .../scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala index 3d34b15d6784a..cd0d3ba8efaa3 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala @@ -406,7 +406,7 @@ object JavaTypeInference { expressions.Literal(fieldName) :: extractorFor(fieldValue, fieldType) :: Nil }) } else { - throw new UnsupportedOperationException(s"Cannot infer type for Java class ${other.getName} because it is not bean-compliant}") + throw new UnsupportedOperationException(s"Cannot infer type for Java class ${other.getName} because it is not bean-compliant") } } } From a85f52b0f5c2acaef78dce9a9730cb79456f9c81 Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 21 Jan 2016 20:16:15 -0700 Subject: [PATCH 3/3] fix style issue - line was over 100 chars --- .../org/apache/spark/sql/catalyst/JavaTypeInference.scala | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala index cd0d3ba8efaa3..3c3717d5043aa 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala @@ -406,7 +406,8 @@ object JavaTypeInference { expressions.Literal(fieldName) :: extractorFor(fieldValue, fieldType) :: Nil }) } else { - throw new UnsupportedOperationException(s"Cannot infer type for Java class ${other.getName} because it is not bean-compliant") + throw new UnsupportedOperationException( + s"Cannot infer type for class ${other.getName} because it is not bean-compliant") } } }