Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/8984#discussion_r41694502
--- Diff: R/pkg/R/DataFrame.R ---
@@ -1880,4 +1880,46 @@ setMethod("as.data.frame",
stop(paste("Unused argument(s): ", paste(list(...),
collapse=", ")))
}
collect(x)
+ }
+)
+
+#' Returns the column types of a DataFrame.
+#'
+#' @name coltypes
+#' @title Get column types of a DataFrame
+#' @param x (DataFrame)
+#' @return value (character) A character vector with the column types of
the given DataFrame
+#' @rdname coltypes
+setMethod("coltypes",
+ signature(x = "DataFrame"),
+ function(x) {
+ # TODO: This may be moved as a global parameter
+ # These are the supported data types and how they map to
+ # R's data types
+ DATA_TYPES <- c("string"="character",
+ "long"="integer",
+ "tinyint"="integer",
+ "short"="integer",
+ "integer"="integer",
+ "byte"="integer",
+ "double"="numeric",
+ "float"="numeric",
+ "decimal"="numeric",
+ "boolean"="logical"
+ )
--- End diff --
You only handle primitive types here, but no complex types, like Array,
Struct and Map.
It would be better you can refactor the type mapping related code here and
that in SerDe.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
