wjones127 commented on a change in pull request #12536:
URL: https://github.com/apache/arrow/pull/12536#discussion_r832736100



##########
File path: ci/appveyor-cpp-setup.bat
##########
@@ -115,3 +115,17 @@ powershell.exe -Command "Start-Process clcache-server" || 
exit /B
 if "%ARROW_S3%" == "ON" (
     appveyor DownloadFile 
https://dl.min.io/server/minio/release/windows-amd64/minio.exe -FileName 
C:\Windows\Minio.exe || exit /B
 )
+
+
+@rem
+@rem Download IANA Timezone Database for unit tests
+@rem
+@rem (Doc section: Download timezone database)
+curl https://data.iana.org/time-zones/releases/tzdata2021e.tar.gz --output 
tzdata.tar.gz

Review comment:
       No that `2021e` is a version, which won't necessarily align with the R 
and Python ones in the future. I don't think it matters that we update it, 
unless the format changes somehow. This is just for testing that it works, and 
we don't ship it.
   
   But the R unit tests use the one provided by the tzdb package, so we are 
testing that as well.




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


Reply via email to