You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/10/06 07:52:44 UTC

[GitHub] [arrow] romainfrancois commented on a change in pull request #8341: ARROW-10093: [R] Add ability to opt-out of int64 -> int demotion

romainfrancois commented on a change in pull request #8341:
URL: https://github.com/apache/arrow/pull/8341#discussion_r500074475



##########
File path: r/src/array_to_vector.cpp
##########
@@ -960,6 +960,14 @@ bool ArraysCanFitInteger(ArrayVector arrays) {
   return all_can_fit;
 }
 
+bool option_arrow_disable_int64_auto_conversion() {

Review comment:
       Went for `bool GetBoolOption(const std::string& name, bool default_)` so that we can specify what the default is when option is not set. 




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org