rok commented on PR #49353: URL: https://github.com/apache/arrow/pull/49353#issuecomment-3998148568
Sorry for slow reply @canassa. Thanks for checking if we can avoid changing the vendored lib! We had recently moved to use `std::chrono` for accessing timezone data on Windows https://github.com/apache/arrow/pull/48601 circumventing the vendored library and need to deal with timezone files. If you're on Windows, could you test current main branch? (current release doesn't have this yet) If you're still seeing this issue, it would be more productive to disable vendored library further and switch to `std::chrono` than to keep hardcoding `TZDIR`. If it'd still make sense to keep with currently proposed approach, then - the JNI CI failures could be related to your change. Can you please rebase this branch so we can check with CI? -- 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]
