jonkeane commented on PR #48601: URL: https://github.com/apache/arrow/pull/48601#issuecomment-3892763417
> @jonkeane in light of CRAN build issues are we ok to merge here? I wish we had a way of testing more reliably on the older macOS framework. What's in https://github.com/apache/arrow/pull/49221#discussion_r2793572830 I'm not really comfortable merging to main, but does exercise the code path. There's at least one other place we need to clean up on main already https://github.com/apache/arrow/pull/49223 so maybe it's fine to merge this and when we get a better way to test this manualy or in CI we'll need to revert this or add a work around. I don't want worries about CRAN to totally slow down everytihng C++(20) related, but there is a bit of an annoying place we are in! -- 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]
