Skip to content
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

Changes for SQL Server support #13

Closed
wants to merge 7 commits into from

Conversation

tsegismont
Copy link

@tsegismont tsegismont commented Mar 31, 2021

Please review each commit separetely.

Vert.x Snapshots are published to the Sonatype's new infra for OSS projects.

This allow to run tests with, for example '-PenableSonatypeOpenSourceSnapshotsRep -PvertxVersion=4.1.0-SNAPSHOT'
@DavideD
Copy link
Owner

DavideD commented Apr 6, 2021

Thanks, I will try this soon

SQL Server expects variables/parameters to begin with the [at] sign
@tsegismont tsegismont changed the title Added enableSonatypeOpenSourceSnapshotsRep property Make NaturalIdTest pass with SQL Server Apr 30, 2021
@DavideD
Copy link
Owner

DavideD commented Apr 30, 2021

Hey @tsegismont, sorry, we had some issues on reactive and I hadn't had the time to look into this. Now that we have release Reactive CR4 we should be able to get back to it. Thanks a lot for the help.

@tsegismont
Copy link
Author

tsegismont commented Apr 30, 2021 via email

Fixes hibernate#743

SQL Server understands neither temporary nor local/global keywords.
Instead, it expects temporary local table names to begin with '#", global with '##'.
@DavideD
Copy link
Owner

DavideD commented May 25, 2021

THanks @tsegismont !

I've rebased this on a new branch that I will push on hibernate/hibernate-reactive or merge if ready
(I still haven't checked all the the changes).

@DavideD DavideD closed this May 25, 2021
@tsegismont
Copy link
Author

@DavideD great! Which is the new branch?

@DavideD
Copy link
Owner

DavideD commented May 25, 2021

I haven't pushed it yet upstream as I'm checking that everything is alright.
Gimme 10 minutes :-)

@DavideD
Copy link
Owner

DavideD commented May 25, 2021

@tsegismont tsegismont deleted the mssql-vertx-4 branch July 2, 2021 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants