kou commented on PR #43094: URL: https://github.com/apache/arrow/pull/43094#issuecomment-2212037371
> The library is part of the standard for c++ 20 and some compilers now, so I think the proper fix would be not to use it anymore. How can we do this? Does `using arrow_vendored::date::year_month_day = std::chrono::year_month_day;` or something work? > Do you think it's good enough to handle it like this for now? What does "this" refer? Changing our bundled date library directly? If so, it's not acceptable. We don't want to maintain these changes in our side. We should upstream these changes. -- 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]
