kou commented on code in PR #14105:
URL: https://github.com/apache/arrow/pull/14105#discussion_r969656023


##########
.github/workflows/cpp.yml:
##########
@@ -271,8 +271,8 @@ jobs:
         uses: actions/cache@v2
         with:
           path: ${{ steps.ccache-info.outputs.cache-dir }}
-          key: cpp-ccache-windows-${{ hashFiles('cpp/**') }}
-          restore-keys: cpp-ccache-windows-
+          key: cpp-ccache-windows-v2-${{ hashFiles('cpp/**') }}
+          restore-keys: cpp-ccache-windows-v2-

Review Comment:
   
https://github.com/apache/arrow/actions/runs/3043600576/jobs/4905543765#step:11:1405
 ?
   
   ```text
   FAILED: debug/arrow-public-api-test.exe 
   cmd.exe /C "cd . && "C:\Program Files\CMake\bin\cmake.exe" -E vs_link_exe 
--intdir=src\arrow\CMakeFiles\arrow-public-api-test.dir 
--rc=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\rc.exe 
--mt=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\mt.exe --manifests  -- 
C:\PROGRA~2\MICROS~1\2019\ENTERP~1\VC\Tools\MSVC\1429~1.301\bin\Hostx64\x64\link.exe
 /nologo src\arrow\CMakeFiles\arrow-public-api-test.dir\public_api_test.cc.obj  
/out:debug\arrow-public-api-test.exe /implib:debug\arrow-public-api-test.lib 
/pdb:debug\arrow-public-api-test.pdb /version:0.0 /machine:x64  
/NODEFAULTLIB:LIBCMT /debug /INCREMENTAL /subsystem:console  
debug\arrow_testing.lib  debug\arrow.lib  "C:\Program 
Files\OpenSSL-Win64\lib\VC\libcrypto64MDd.lib"  "C:\Program 
Files\OpenSSL-Win64\lib\VC\libssl64MDd.lib"  
utf8proc_ep-install\lib\utf8proc_static.lib  re2_ep-install\lib\re2.lib  
googletest_ep-prefix\lib\gtest_maind.lib  googletest_ep-prefix\lib\gtestd.lib  
googletest_ep-prefix\lib\gmockd.lib  "C:\Program Files\OpenSSL
 -Win64\lib\VC\libcrypto64MDd.lib"  
mimalloc_ep\src\mimalloc_ep\lib\mimalloc-2.0\mimalloc-static-debug.lib  
bcrypt.lib  psapi.lib  ws2_32.lib  kernel32.lib user32.lib gdi32.lib 
winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib 
advapi32.lib && cd ."
   LINK Pass 1: command 
"C:\PROGRA~2\MICROS~1\2019\ENTERP~1\VC\Tools\MSVC\1429~1.301\bin\Hostx64\x64\link.exe
 /nologo src\arrow\CMakeFiles\arrow-public-api-test.dir\public_api_test.cc.obj 
/out:debug\arrow-public-api-test.exe /implib:debug\arrow-public-api-test.lib 
/pdb:debug\arrow-public-api-test.pdb /version:0.0 /machine:x64 
/NODEFAULTLIB:LIBCMT /debug /INCREMENTAL /subsystem:console 
debug\arrow_testing.lib debug\arrow.lib C:\Program 
Files\OpenSSL-Win64\lib\VC\libcrypto64MDd.lib C:\Program 
Files\OpenSSL-Win64\lib\VC\libssl64MDd.lib 
utf8proc_ep-install\lib\utf8proc_static.lib re2_ep-install\lib\re2.lib 
googletest_ep-prefix\lib\gtest_maind.lib googletest_ep-prefix\lib\gtestd.lib 
googletest_ep-prefix\lib\gmockd.lib C:\Program 
Files\OpenSSL-Win64\lib\VC\libcrypto64MDd.lib 
mimalloc_ep\src\mimalloc_ep\lib\mimalloc-2.0\mimalloc-static-debug.lib 
bcrypt.lib psapi.lib ws2_32.lib kernel32.lib user32.lib gdi32.lib winspool.lib 
shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.li
 b /MANIFEST 
/MANIFESTFILE:src\arrow\CMakeFiles\arrow-public-api-test.dir/intermediate.manifest
 src\arrow\CMakeFiles\arrow-public-api-test.dir/manifest.res" failed (exit code 
1120) with the following output:
   public_api_test.cc.obj : error LNK2019: unresolved external symbol 
"__declspec(dllimport) class std::optional<class std::basic_string<char,struct 
std::char_traits<char>,class std::allocator<char> > > __cdecl 
arrow::GetTestTimezoneDatabaseRoot(void)" 
(__imp_?GetTestTimezoneDatabaseRoot@arrow@@YA?AV?$optional@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@std@@XZ)
 referenced in function "private: virtual void __cdecl 
arrow::Misc_SetTimezoneConfig_Test::TestBody(void)" 
(?TestBody@Misc_SetTimezoneConfig_Test@arrow@@EEAAXXZ)
   debug\arrow-public-api-test.exe : fatal error LNK1120: 1 unresolved externals
   ```
   
   I agree with you.
   
   How about the following for easy to understand this?
   
   ```diff
   diff --git a/.github/workflows/cpp.yml b/.github/workflows/cpp.yml
   index d4c5c4c897..4b35e8365f 100644
   --- a/.github/workflows/cpp.yml
   +++ b/.github/workflows/cpp.yml
   @@ -271,8 +271,11 @@ jobs:
            uses: actions/cache@v2
            with:
              path: ${{ steps.ccache-info.outputs.cache-dir }}
   -          key: cpp-ccache-windows-${{ hashFiles('cpp/**') }}
   -          restore-keys: cpp-ccache-windows-
   +          key: cpp-ccache-windows-${{ env.CACHE_VERSION }}-${{ 
hashFiles('cpp/**') }}
   +          restore-keys: cpp-ccache-windows-${{ env.CACHE_VERSION }}-
   +        env:
   +          # We can invalidate the current cache by updating this.
   +          CACHE_VERSION: "2022-09-13"
          - name: Build
            shell: cmd
            run: |
   ```



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