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

Posted to github@arrow.apache.org by "mikelui (via GitHub)" <gi...@apache.org> on 2023/04/15 14:48:34 UTC

[GitHub] [arrow] mikelui commented on pull request #35135: GH-35134: [C++] Add `arrow_vendored` namespace around double-conversion library

mikelui commented on PR #35135:
URL: https://github.com/apache/arrow/pull/35135#issuecomment-1509860513

   > Could you wait for a response from Gandiva developers? [#34919 (comment)](https://github.com/apache/arrow/pull/34919#issuecomment-1501420706)
   
   This doesn't change the current status-quo of the patched changes, and instead just makes it explicit. Would it make sense to merge this now, and allow for the upstream changes for the version update?
   
   The patch in the script would actually fail since those files don't exist in the newest version of double-conversion (the version being updated in that PR)
   
   


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