Skip to content

fix: unexpected results when embedding the same table twice#4104

Merged
wolfgangwalther merged 1 commit into
PostgREST:mainfrom
laurenceisla:fix/alias-flt
Apr 29, 2026
Merged

fix: unexpected results when embedding the same table twice#4104
wolfgangwalther merged 1 commit into
PostgREST:mainfrom
laurenceisla:fix/alias-flt

Conversation

@laurenceisla
Copy link
Copy Markdown
Member

@laurenceisla laurenceisla commented May 27, 2025

Closes #4075

(In stand-by for a major release)

Comment thread test/spec/Feature/Query/QuerySpec.hs
@steve-chavez steve-chavez added the breaking change A bug fix or enhancement that would cause a breaking change label May 28, 2025
themavik

This comment was marked as spam.

@wolfgangwalther
Copy link
Copy Markdown
Member

(In stand-by for a major release)

Since we only have major and patch releases by now, there is no need to hold this back. Also we already have some breaking changes on main anyway.

@laurenceisla laurenceisla marked this pull request as ready for review April 27, 2026 15:41
@laurenceisla laurenceisla force-pushed the fix/alias-flt branch 2 times, most recently from 71911c7 to 0143767 Compare April 27, 2026 20:33
Copy link
Copy Markdown
Member

@wolfgangwalther wolfgangwalther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebased to resolve the conflict in CHANGELOG.

@wolfgangwalther wolfgangwalther merged commit 5f6f7dc into PostgREST:main Apr 29, 2026
47 of 53 checks passed
@laurenceisla laurenceisla deleted the fix/alias-flt branch April 29, 2026 16:14
@steve-chavez
Copy link
Copy Markdown
Member

steve-chavez commented May 9, 2026

@laurenceisla @wolfgangwalther This shouldn't haven been merged in this state.

  • This syntax is very old and the change very subtle.
  • We have a big amount of deployments now that have lots of requests over their application logic.
  • We can't expect users to hunt cases where this happens. It makes the migration very hard.

I agree the change should be done but gradually. I'll push a PR shortly. Edit: done #4905

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

Labels

breaking change A bug fix or enhancement that would cause a breaking change

Development

Successfully merging this pull request may close these issues.

Unexpected results when embedding and filtering the same table twice

4 participants