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 2021/06/01 19:29:12 UTC

[GitHub] [arrow] kszucs commented on pull request #10414: [C++]: C++17 test PR

kszucs commented on pull request #10414:
URL: https://github.com/apache/arrow/pull/10414#issuecomment-852389362


   > Debian 9 ([builds JNI](https://github.com/apache/arrow/pull/10414/checks?check_run_id=2687677911#step:8:2269)) has only gcc 6.3, which doesn't have `<string_view>`. https://wandbox.org/permlink/bijOi9BHbk3HXqiA In general it seems to lack a number of c++17 features
   
   We should bump the debian version to 10 in those docker images (which is the default in `.env` but hardcoded in the java images). We may need to bump the `JDK` and `mvn` versions as well due to newer debian. If we wan't to keep builds for the current `JDK` and `mvn` versions then we can still parametrize that we test `debian-java` with `DEBIAN=9` and `debian-java-jni` with `DEBIAN=10`. 
   
   I'm going to update the configuration.


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