jonkeane commented on code in PR #48652: URL: https://github.com/apache/arrow/pull/48652#discussion_r2649200788
########## r/R/type.R: ########## @@ -146,13 +146,23 @@ infer_type.Expression <- function(x, ...) x$type() #' @title FixedWidthType class #' +#' @description +#' `FixedWidthType` is a base class for data types with a fixed width in bits. +#' This includes all integer types, floating-point types, `Boolean`, +#' `FixedSizeBinary`, temporal types (dates, times, timestamps, durations), +#' and decimal types. Review Comment: Should we mark this as internal with `#' @keywords internal` or at the least add a note in here that this is basically an internal type, not something that users of Arrow should need? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
