Skip to content

Use default Rails prepared statements - #1775

Open
amaksimo wants to merge 1 commit into
mainfrom
fix/rails-prepared-statements
Open

amaksimo wants to merge 1 commit into
mainfrom
fix/rails-prepared-statements

Conversation

@amaksimo

@amaksimo amaksimo commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • use Rails' standard prepared statement behavior in both Rails samples
  • keep database configuration focused on authentication, TLS, and advisory locks
  • cover the connection setting in both Rails integration suites

Testing

  • Petclinic integration suite passed against Aurora DSQL
  • Car Rental integration suite passed against Aurora DSQL

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT-0 license.

Comment thread ruby/rails/README.md
host: <%= ENV['CLUSTER_ENDPOINT'] %>
# Disable prepared statements and advisory locks for Aurora DSQL
prepared_statements: false
# Disable advisory locks for Aurora DSQL

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nit: probably unnecessary comment

host: <%= ENV['CLUSTER_ENDPOINT'] %>
# Disable prepared statements and advisory locks for Aurora DSQL
prepared_statements: false
# Disable advisory locks for Aurora DSQL

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nit: probably unnecessary comment

This branch has not been deployed

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