romainfrancois commented on a change in pull request #11751:
URL: https://github.com/apache/arrow/pull/11751#discussion_r754335864
##########
File path: r/R/type.R
##########
@@ -40,7 +40,8 @@ DataType <- R6Class("DataType",
fields = function() {
DataType__fields(self)
},
- export_to_c = function(ptr) ExportType(self, ptr)
+ export_to_c = function(ptr) ExportType(self, ptr),
+ code = function() call2("unknown_type", self$ToString())
Review comment:
There are a bunch of types we don't have R functions for, e.g.
`DECIMAL256`, `MAP` ... but then I'm not sure how I would test it, because I
would need to get one of them in the first place ...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
