Re: [I] [C++] Bump bundled gRPC [arrow]

2025-12-17 Thread via GitHub


alinaliBQ commented on issue #46016:
URL: https://github.com/apache/arrow/issues/46016#issuecomment-3666419444

   Thanks  @raulcd for the update, appreciate it. Looking forward to your PR. 
Hopefully folks in the community will have an idea for the fix on `Crc32c`


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [I] [C++] Bump bundled gRPC [arrow]

2025-12-17 Thread via GitHub


raulcd commented on issue #46016:
URL: https://github.com/apache/arrow/issues/46016#issuecomment-3664329165

   With the recent changes to move from `ExternalProject` to `FetchContent` for 
a bunch of our third party dependencies we tried to simplify the version bumps 
as this proved challenging during a long time.
   I did a quick test locally a couple days ago but started having issues of 
targets redefinition between `Crc32c` and `absl::crc32c`. abseil newest 
versions define a `Crc32c` target too and wasn't unable to fix it.
   I might submit a PR "soon-ish" at least sharing the problem in case someone 
has ideas on how to fix it. I am not sure I will be able to do it before new 
year and the 23.0.0 release but just wanted to share that I was stuck on that.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [I] [C++] Bump bundled gRPC [arrow]

2025-12-16 Thread via GitHub


alinaliBQ commented on issue #46016:
URL: https://github.com/apache/arrow/issues/46016#issuecomment-3662474710

   +1 for bumping bundled gRPC. Newer version of gRPC after 1.46.3 includes 
critical fixes to issues such as https://github.com/grpc/grpc/issues/35511. If 
someone can help with this it would be great.
   
   Using gRPC version 1.46.3, I ran into pem root certs access issue on macOS 
Intel which blocks connection to remote server instances that need a cert to 
connect. I do not see this issue using dynamic gRPC library v1.76.0. And the 
short-term workaround is to run
   ```
   export GRPC_DEFAULT_SSL_ROOTS_FILE_PATH=/etc/ssl/cert.pem
   ```
   in case anyone sees the same issue.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [I] [C++] Bump bundled gRPC [arrow]

2025-04-04 Thread via GitHub


kou commented on issue #46016:
URL: https://github.com/apache/arrow/issues/46016#issuecomment-2774850489

   Thanks for the information!
   It seems that we need to bump Abseil before we bump gRPC. (Or we may need to 
bump them at once.)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



Re: [I] [C++] Bump bundled gRPC [arrow]

2025-04-03 Thread via GitHub


raulcd commented on issue #46016:
URL: https://github.com/apache/arrow/issues/46016#issuecomment-2774804918

   I think we have had several attempts of this in the past:
   - https://github.com/apache/arrow/issues/45063
   
   If I recall correctly bumping gRPC requires us to bump Abseil too.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]