search

apache / arrow-adbc

Database connectivity API standard and libraries for Apache Arrow
https://arrow.apache.org/adbc/
Apache License 2.0
378 stars 94 forks source link

c: explicitly set C++11 for non-test code for R's benefit #815

Closed lidavidm closed 1 year ago

lidavidm commented 1 year ago

Ideally we'd be able to switch to C++17 in mid-2024?

paleolimbot commented 1 year ago

Just adding the background here:

There are two platforms that are not uncommon for R where this comes up (R 3.6 on Windows and Centos7/RHEL7 with the default compilers). R 3.6 is fairly old; however, it still shows up on Windows in environments where IT is slow to update software (e.g., some government agencies). RHEL 7 shows up in institution-provisioned servers for platforms like RStudio Server. Both of these use-cases frequently involve end-users that can't do much about their environment although typically they can install R packages from CRAN.

It's not absolutely essential that ADBC database drivers work on these somewhat ancient platforms for R packages; however, if possible it would be nice when I'm promoting the initial versions of the drivers to support all the versions in the tidyverse support grid ( https://www.tidyverse.org/blog/2019/04/r-version-support/ ). The current schedule suggests that 2024 is when support for R 3.6 will be dropped (which I think is also EOL for Centos 7).