Skip to content

Conversation

@the-glu
Copy link
Contributor

@the-glu the-glu commented Jul 14, 2025

Description

This PR add an option to set statement timeout for database drop of the tempdb.

Motivation

When using pg-schema-diff on an azure managed database, the database drop statement take time (around 10s), generating error when using pg-schema-diff (and creating strange shadow databases).

With this, any suitable timeouts can be set depending on user context.

Testing

Tested locally in my context, 'issue' is fixed.

No unit test because it seems hard to test, but I'm open to suggestions.

Copy link
Collaborator

@bplunkett-stripe bplunkett-stripe left a comment

Choose a reason for hiding this comment

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

LGTM! I could imagine folks --might-- want this as a CLI parameter in the future.

@bplunkett-stripe bplunkett-stripe merged commit 8613492 into stripe:main Jul 15, 2025
8 checks passed
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