thisisnic commented on a change in pull request #12474:
URL: https://github.com/apache/arrow/pull/12474#discussion_r816168255
##########
File path: r/R/csv.R
##########
@@ -61,8 +61,8 @@
#' * "l": `bool()`
#' * "f": `dictionary()`
#' * "D": `date32()`
-#' * "T": `timestamp()`
-#' * "t": `time32()`
+#' * "T": `timestamp(unit = "ns")`
+#' * "t": `time32(unit = "ms")`
Review comment:
OK, fine to update it but add a comment after pointing out this is the
default value, so there's no ambiguity. And let's leave it as it is (without
the default option explicitly stated) in the code.
--
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
