Thanks for the catch. Could you send a PR with this diff ?

On Fri, Aug 14, 2015 at 10:30 AM, Shkurenko, Alex <ashkure...@enova.com> wrote:
> Got an issue similar to https://issues.apache.org/jira/browse/SPARK-8897,
> but with the Decimal datatype coming from a Postgres DB:
>
> //Set up SparkR
>
>>Sys.setenv(SPARK_HOME="/Users/ashkurenko/work/git_repos/spark")
>>Sys.setenv(SPARKR_SUBMIT_ARGS="--driver-class-path
>> ~/Downloads/postgresql-9.4-1201.jdbc4.jar sparkr-shell")
>>.libPaths(c(file.path(Sys.getenv("SPARK_HOME"), "R", "lib"), .libPaths()))
>>library(SparkR)
>>sc <- sparkR.init(master="local")
>
> // Connect to a Postgres DB via JDBC
>>sqlContext <- sparkRSQL.init(sc)
>>sql(sqlContext, "
>     CREATE TEMPORARY TABLE mytable
>     USING org.apache.spark.sql.jdbc
>     OPTIONS (url 'jdbc:postgresql://servername:5432/dbname'
>     ,dbtable 'mydbtable'
> )
> ")
>
> // Try pulling a Decimal column from a table
>>myDataFrame <- sql(sqlContext,("select a_decimal_column  from mytable "))
>
> // The schema shows up fine
>
>>show(myDataFrame)
>
> DataFrame[a_decimal_column:decimal(10,0)]
>
>>schema(myDataFrame)
>
> StructType
> |-name = "a_decimal_column", type = "DecimalType(10,0)", nullable = TRUE
>
> // ... but pulling data fails:
>
> localDF <- collect(myDataFrame)
>
> Error in as.data.frame.default(x[[i]], optional = TRUE) :
>   cannot coerce class ""jobj"" to a data.frame
>
>
> -------
> Proposed fix:
>
> diff --git a/core/src/main/scala/org/apache/spark/api/r/SerDe.scala
> b/core/src/main/scala/org/apache/spark/api/r/SerDe.scala
> index d5b4260..b77ae2a 100644
> --- a/core/src/main/scala/org/apache/spark/api/r/SerDe.scala
> +++ b/core/src/main/scala/org/apache/spark/api/r/SerDe.scala
> @@ -219,6 +219,9 @@ private[spark] object SerDe {
>          case "float" | "java.lang.Float" =>
>            writeType(dos, "double")
>            writeDouble(dos, value.asInstanceOf[Float].toDouble)
> +        case "decimal" | "java.math.BigDecimal" =>
> +           writeType(dos, "double")
> +           writeDouble(dos,
> scala.math.BigDecimal(value.asInstanceOf[java.math.BigDecimal]).toDouble)
>          case "double" | "java.lang.Double" =>
>            writeType(dos, "double")
>            writeDouble(dos, value.asInstanceOf[Double])
>
> Thanks,
> Alex

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@spark.apache.org
For additional commands, e-mail: dev-h...@spark.apache.org

Reply via email to