Yicong-Huang commented on code in PR #6049:
URL: https://github.com/apache/texera/pull/6049#discussion_r3510385699


##########
.github/workflows/build.yml:
##########
@@ -281,6 +281,8 @@ jobs:
         # those run in the amber-integration job below.
         env:
           AMBER_TEST_FILTER: skip-integration
+          # unit job uses its provisioned postgres catalog; default (rest) 
needs a Lakekeeper not run here
+          STORAGE_ICEBERG_CATALOG_TYPE: postgres

Review Comment:
   I see. there are more cleaning  to do. in this case, let's table it.
   
   let's keep those test in amber for now, leave postgres in amber,  change 
default to rest.  
   
   gradually we can move those tests to intergration, and clean up amber CI 
later.



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