Skip to content

fix: request failures when work_mem is set on a role#4977

Merged
steve-chavez merged 1 commit into
PostgREST:mainfrom
laurenceisla:fix/lowcase-pg-config
Jun 3, 2026
Merged

fix: request failures when work_mem is set on a role#4977
steve-chavez merged 1 commit into
PostgREST:mainfrom
laurenceisla:fix/lowcase-pg-config

Conversation

@laurenceisla

Copy link
Copy Markdown
Member

Closes #4955

Comment thread src/PostgREST/Config/Database.hs Outdated
Comment thread src/PostgREST/Config/Database.hs
@laurenceisla laurenceisla force-pushed the fix/lowcase-pg-config branch from f035e7d to 0cbd5df Compare June 3, 2026 17:10
@laurenceisla laurenceisla force-pushed the fix/lowcase-pg-config branch from 0cbd5df to 0719327 Compare June 3, 2026 17:15
@steve-chavez

Copy link
Copy Markdown
Member

@laurenceisla Weird failure on pg 16 https://github.com/PostgREST/postgrest/actions/runs/26900983802/job/79352994410?pr=4977

@laurenceisla

Copy link
Copy Markdown
Member Author

@laurenceisla Weird failure on pg 16 https://github.com/PostgREST/postgrest/actions/runs/26900983802/job/79352994410?pr=4977

I rerun and it looks like it's working now. Maybe transient? (schema cache reload timing and stuff)

@steve-chavez

Copy link
Copy Markdown
Member

I rerun and it looks like it's working now. Maybe transient? (schema cache reload timing and stuff)

Right, it was this test first time seeing it fail randomly:

def test_schema_cache_concurrent_notifications(slow_schema_cache_env):

#4962 might help with that.

@steve-chavez steve-chavez merged commit 416a154 into PostgREST:main Jun 3, 2026
47 of 48 checks passed
@postgrest-ci

postgrest-ci Bot commented Jun 3, 2026

Copy link
Copy Markdown

Backport failed for v14, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin v14
git worktree add -d .worktree/backport-4977-to-v14 origin/v14
cd .worktree/backport-4977-to-v14
git switch --create backport-4977-to-v14
git cherry-pick -x 416a15480f32c592482284318985c0c758814b8b

@steve-chavez

Copy link
Copy Markdown
Member

@taimoorzaeem Could you help backporting this one? 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Setting work_mem on a role causes request failure

2 participants