Yicong-Huang opened a new pull request, #4633:
URL: https://github.com/apache/texera/pull/4633

   ### What changes were proposed in this PR?
   
   Switch the python job in `build.yml` from `apt-get install postgresql` + 
`systemctl start` to a `services: postgres` container, mirroring what the scala 
job already does:
   
   - Add `services.postgres` (image `postgres`, `POSTGRES_PASSWORD=postgres`, 
port 5432, `pg_isready` healthcheck).
   - Drop `Install PostgreSQL`, `Start PostgreSQL Service`, and the `sudo -u 
postgres psql -f` seed step.
   - Single `Create iceberg catalog database` step that runs `psql -h localhost 
-U postgres -f sql/iceberg_postgres_catalog.sql` (same pattern as the scala 
job).
   
   ### Any related issues, documentation, discussions?
   
   Driven by repeated python-job failures on `apt-get update` against 
`azure.archive.ubuntu.com`, which has been unreliable; runs sit ignoring the 
InRelease responses for tens of seconds and either fail or surface stale 
package metadata. The docker registry path used by `services` is independent of 
that mirror.
   
   Side benefit: postgres container starts in seconds, vs. ~30 s of `apt-get 
update` even on a healthy day. Removes the only place in `build.yml` that still 
needed the apt mirror.
   
   ### How was this PR tested?
   
   Will be exercised by this PR's own python matrix once the CI runs. The seed 
SQL is the same one the scala job already runs successfully against the same 
container image.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Opus 4.7
   


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