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]