chore: fix and enable pljava #690
Conversation
pcnc
force-pushed
the
pcnc/fix-pljava
branch
2 times, most recently
from
1c3bc6e to
2720623
Compare
| @@ -1,3 +1,3 @@ | |||
| BEGIN; | |||
| create extension if not exists pljava with schema "sqlj"; | |||
There was a problem hiding this comment.
pljava creates the sqlj schema automatically
| @@ -80,11 +82,12 @@ jobs: | |||
| ubuntu_release_no=${{ matrix.ubuntu_version }} | |||
| postgresql_major=${{ steps.version.outputs.postgresql_major }} | |||
| postgresql_release=${{ steps.version.outputs.postgresql_release }} | |||
| CPPFLAGS=-mcpu=${{ matrix.mcpu }} | |||
| CPPFLAGS= | |||
| DEB_BUILD_PROFILES=pkg.postgresql.nozstd | |||
There was a problem hiding this comment.
Building Postgres from source requires libzstd 1.4.0+, whereas Ubuntu 18 only has versions up to 1.3.3 available.
Sadness all around, although it doesn't affect us since these binaries are only used to upgrade from PG13/14 (which didn't have zstd compression support)
| @@ -46,6 +46,8 @@ jobs: | |||
| - uses: docker/setup-buildx-action@v2 | |||
| with: | |||
| endpoint: builders | |||
| driver-opts: image=moby/buildkit:v0.11.6 | |||
There was a problem hiding this comment.
Updated buildkit just in case it'd help us avoid connection reset errors; default was running under v0.10.0
| @@ -56,7 +58,7 @@ jobs: | |||
| tags: supabase/postgres:extensions | |||
| platforms: linux/${{ matrix.arch }} | |||
| cache-from: type=gha | |||
| cache-to: type=gha,mode=max | |||
| cache-to: type=gha,mode=max,ignore-error=true | |||
There was a problem hiding this comment.
Also no point crashing the entire workflow if the build cache push fails
| ADD --checksum=${pljava_release_checksum} \ | ||
| "https://github.com/supabase/pljava/archive/refs/heads/${pljava_release}.tar.gz" \ | ||
| "https://github.com/tada/pljava/archive/refs/tags/V${pljava_release}.tar.gz" \ |
There was a problem hiding this comment.
Reverting to original repo since PG15 support was merged in the meantime
…ss` when building the .deb for pljava Co-authored-by: Han Qiao <qiao@supabase.io>
Not yet - this is pending today - upgrades breaking were the main driver for this PR |
Uh oh!
There was an error while loading. Please reload this page.