Skip to content

Conversation

@sergiyvamz
Copy link
Contributor

@sergiyvamz sergiyvamz commented Oct 15, 2025

Summary

chore: refactoring pg sql queries to be fully qualified

Additional Reviewers

@karenc-bq
@aaron-congo

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sergiyvamz sergiyvamz changed the title chore: refactoring sql queries to be fully qualified chore: refactoring pg sql queries to be fully qualified Oct 15, 2025
@aaron-congo
Copy link
Contributor

Need to also update a call to aurora_limitless_router_endpoints in our messages.properties file

@aaron-congo
Copy link
Contributor

Do any of the following need to be prefixed by "pg_catalog" as well? Some of these appear to be in pg_catalog but I'm not sure if they need it

  • coalesce()
  • extract()
  • from()
  • operator()
  • substring()
  • position()
  • max()
  • split_part()
  • count()

@sergiyvamz
Copy link
Contributor Author

Do any of the following need to be prefixed by "pg_catalog" as well? Some of these appear to be in pg_catalog but I'm not sure if they need it

  • coalesce()
  • extract()
  • from()
  • operator()
  • substring()
  • position()
  • max()
  • split_part()
  • count()

It wasn't planned. Some of those functions are built-in so requires no pg_catalog.

@sergiyvamz sergiyvamz merged commit b62183b into main Oct 16, 2025
6 checks passed
@sergiyvamz sergiyvamz deleted the fix-qualify-queries branch October 16, 2025 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants