[jira] [Commented] (SPARK-6290) spark.ml.param.Params.checkInputColumn bug upon error

["Glenn Weidner (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SPARK-6290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14508246#comment-14508246 ] 

Glenn Weidner commented on SPARK-6290:
--------------------------------------

I would like to work on this.  In a test scenario created to force the error, observed the first part of the message was helpful in describing the failure due to types not matching and included the required vs. actual type information.  However, the second part of the message contained a java.lang.IllegalArgumentException embedded with stack trace.  For example:

java.lang.IllegalArgumentException: requirement failed: 
	at scala.Predef$.require(Predef.scala:233)
	at org.apache.spark.ml.param.Params$class.checkInputColumn(params.scala:206)

Can you confirm the exception stack trace part is the "weird" error referred to in the description?  Thanks.

> spark.ml.param.Params.checkInputColumn bug upon error
> -----------------------------------------------------
>
>                 Key: SPARK-6290
>                 URL: https://issues.apache.org/jira/browse/SPARK-6290
>             Project: Spark
>          Issue Type: Bug
>          Components: ML
>    Affects Versions: 1.3.0
>            Reporter: Joseph K. Bradley
>            Priority: Minor
>
> In checkInputColumn, if data types do not match, it tries to print an error message with this in it:
> {code}
> Column param description: ${getParam(colName)}"
> {code}
> However, getParam cannot be called on the string colName; it needs the parameter name, which this method is not given.  This causes a weird error which users may find hard to understand.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org