[SPARK-12932] [Java API] improved error message for java type inference failure

[andygrove]

No description provided.

[srowen]

(Link this to your JIRA -- see guidance here first for how to open a PR: https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark )

[SparkQA]

Test build #2432 has finished for PR 10865 at commit f11f1c7.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • throw new UnsupportedOperationException(s\"Cannot infer type for Java class $

[rxin]

cc @cloud-fan

[rxin]

this needs to be less than 100 chars wide to fit spark's style guide

[andygrove]

Thanks. I have pushed a fix.

On Thu, Jan 21, 2016 at 8:12 PM, Reynold Xin [email protected]
wrote:

In
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala
#10865 (comment):

@@ -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")
  

this needs to be less than 100 chars wide to fit spark's style guide

—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/10865/files#r50498058.

[cloud-fan]

LGTM

[SparkQA]

Test build #2436 has finished for PR 10865 at commit a85f52b.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[SparkQA]

Test build #2440 has finished for PR 10865 at commit a85f52b.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[SparkQA]

Test build #2441 has finished for PR 10865 at commit a85f52b.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[SparkQA]

Test build #2442 has finished for PR 10865 at commit a85f52b.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[rxin]

Looks like there are some test failures?

[SparkQA]

Test build #2444 has finished for PR 10865 at commit a85f52b.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[srowen]

Yeah, these are consistent test failures but they're not due to this change.

[cloud-fan]

looks like it's related to the jenkins machine that run this test?

[JoshRosen]

Jenkins, retest this please.

[SparkQA]

Test build #49980 has finished for PR 10865 at commit a85f52b.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • s\"Cannot infer type for class $

[srowen]

Merged to master/1.6