yaooqinn commented on a change in pull request #32037:
URL: https://github.com/apache/spark/pull/32037#discussion_r611707438



##########
File path: .github/workflows/build_and_test.yml
##########
@@ -485,11 +485,10 @@ jobs:
         restore-keys: |
           tpcds-
     - name: Checkout TPC-DS (SF=1) generated data repository
-      if: steps.cache-tpcds-sf-1.outputs.cache-hit != 'true'

Review comment:
       I guess it's based on the catch key here 
https://github.com/apache/spark/pull/32037/files#diff-48c0ee97c53013d18d6bbae44648f7fab9af2e0bf5b0dc1ca761e18ec5c478f2R484,
  which is based on the cache data itself(seems a mistake we made before?), and 
seems that it never changed if we have no opportunity to change the cache data 
entirely by disabling it. Or let me try to modify the key to rely on the 
lastest revision of maropu/tpcds-sf-1 




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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to