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

Add new ERMicrosoftPlugIn supporting PK generation from sequences #984

Conversation

hprange
Copy link
Contributor

@hprange hprange commented May 13, 2022

Improves the support for SQL Server with WebObjects. The ERMicrosoftPlugIn generates primary keys from sequences instead of the EO_PK_TABLE strategy used by the standard MicrosoftPlugIn.

Also, the original implementation of the MicrosoftPlugIn forcibly adds the SelectMethod=cursor parameter to the connection URL. Even though this option may be useful in some scenarios, it may also cause undesirable side effects. Each application may have different requirements regarding how and when to use adaptive buffering. For this reason, the ERMicrosoftPlugin lets the user decide how to configure adaptive buffering if needed.

See the Microsoft documentation for more information about adaptive buffering.

@hprange hprange force-pushed the feature/sql-server-generate-pk-from-sequence branch from a08b4ea to 7d11b5d Compare June 2, 2022 22:24
hprange added 2 commits June 3, 2022 19:30
Improves the support for SQL Server with WebObjects. The `ERMicrosoftPlugIn` generates primary keys from sequences instead of the `EO_PK_TABLE` strategy used by the standard `MicrosoftPlugIn`.
The original implementation of the `MicrosoftPlugIn forcibly adds the `SelectMethod=cursor` parameter to the connection URL. Even though this option may be useful in some scenarios, it may also cause undesirable side effects. Each application may have different requirements regarding how and when to use adaptive buffering. For this reason, the `ERMicrosoftPlugin` lets the user decide how to configure adaptive buffering if needed.

See the Microsoft documentation for more information about [adaptive buffering](https://docs.microsoft.com/en-us/sql/connect/jdbc/using-adaptive-buffering).
@hprange hprange force-pushed the feature/sql-server-generate-pk-from-sequence branch from 7d11b5d to 2db1d55 Compare June 3, 2022 22:32
@hprange hprange marked this pull request as ready for review June 3, 2022 22:36
@hprange hprange changed the title [WIP] Add new ERMicrosoftPlugIn supporting PK generation from sequences Add new ERMicrosoftPlugIn supporting PK generation from sequences Jun 3, 2022
@hprange
Copy link
Contributor Author

hprange commented Jul 24, 2022

I've been using this plug-in for development for a while and haven't found any issues. Since this plug-in is not enabled by default, I'm willing to merge this pull request. I'm working on other improvements to this plug-in, but they deserve a separate pull request. Any objections?

@hprange hprange merged commit a859052 into wocommunity:master Jul 25, 2022
@hprange hprange deleted the feature/sql-server-generate-pk-from-sequence branch July 25, 2022 19:30
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.

1 participant