-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Remove statements not supported by SQLExpress from migration test databases #9929
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9929 +/- ##
==========================================
- Coverage 42.27% 42.27% -0.01%
==========================================
Files 608 608
Lines 79392 79392
==========================================
- Hits 33566 33560 -6
- Misses 41681 41688 +7
+ Partials 4145 4144 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why gzipped SQL scripts?
They were originally gzipped. I just decompressed, edited and recompressed. |
They're gzipped because otherwise they're very large and they're not really code or meant to be updated. |
I'm unable to run
make test-mssql-migrations
tests in my environment, neither with my old SQL Server 2008R2 or with the SQLExpress 15.0.1900.25 I've just installed in my CentOS 7. The test databases were exported from a modern full-fledged version of MSSQL and they contain some commands that make Express fail (I figure there's no point in addressing 2008R2).I've edited the files to remove three statements that were incompatible with Express, so the tests PASS in my system:
AFAIU these statements are defaults and removing them should be innocuous:
PS: I know, I work with very old stuff.... 😢