Skip to content

Always commit when executing DDL#33

Merged
martinburchell merged 2 commits intomasterfrom
execute-ddl-commit
Mar 17, 2025
Merged

Always commit when executing DDL#33
martinburchell merged 2 commits intomasterfrom
execute-ddl-commit

Conversation

@martinburchell
Copy link
Collaborator

This fixes a problem seen when preprocessing SystmOne data on the CPFT CRATE server. If a crate_pk column was added to a table and in the same run an index created on this column, SQL Server would report that this column didn't exist.

This fixes a problem seen when preprocessing SystmOne data on the CPFT server.
If a 'crate_pk' column was added to a table and an index created on this column,
SQL Server would report that this column didn't exist
Copy link
Owner

@RudolfCardinal RudolfCardinal left a comment

Choose a reason for hiding this comment

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

Thank you! And sorry for causing the problem. PyCharm etc. may want "# noqa: E501" at the end of the long URL lines. Content-wise, short and perfect.

@martinburchell
Copy link
Collaborator Author

Thank you! And sorry for causing the problem. PyCharm etc. may want "# noqa: E501" at the end of the long URL lines. Content-wise, short and perfect.

URLs seem to be exempt from that rule, for flake8 at least. I don't know about PyCharm,

@martinburchell martinburchell merged commit f37fda8 into master Mar 17, 2025
1 check passed
@martinburchell martinburchell deleted the execute-ddl-commit branch March 17, 2025 15:36
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