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 2022/02/03 08:30:51 UTC

[GitHub] [arrow] mbrobbel commented on pull request #12322: ARROW-15533: [C++] Check ARROW_WITH_OPENTELEMETRY in CI

mbrobbel commented on pull request #12322:
URL: https://github.com/apache/arrow/pull/12322#issuecomment-1028723684


   > Is the option making it though to the build? I see the env variable but this line suggests it's not actually turned on in the CMake Configure:
   > 
   > https://github.com/apache/arrow/runs/5040342323?check_suite_focus=true#step:7:460
   > 
   
   Thanks for spotting that! I forgot to update the build script.
   
   > Also, maybe we should also add this to a Linux build, like the `AMD64 Conda C++` one?
   
   This is blocked on https://github.com/apache/arrow/pull/11963. I'll add a reminder to that PR to enable OpenTelemetry in CI once that gets merged.


-- 
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-unsubscribe@arrow.apache.org

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