Skip to content

Conversation

@danielfrankcom
Copy link
Contributor

This PR configures a default value for DISABLE_SERVER_SIDE_CURSORS, since server-side cursors are not supported by DSQL.

The default value here helps us avoid requiring customers to configure this in their own project. It also makes it easier for us to enable again if support is changed in the future.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

vic-tsang
vic-tsang previously approved these changes Sep 25, 2025
@danielfrankcom danielfrankcom force-pushed the dev/frankcom/autofield-patch branch from 24585b1 to 2e3f76e Compare September 25, 2025 21:56
Base automatically changed from dev/frankcom/autofield-patch to version-0 September 26, 2025 18:09
@danielfrankcom danielfrankcom dismissed vic-tsang’s stale review September 26, 2025 18:09

The base branch was changed.

@danielfrankcom danielfrankcom force-pushed the dev/frankcom/server-side-cursors branch from c26c511 to e7d8b36 Compare September 26, 2025 18:11
@danielfrankcom
Copy link
Contributor Author

Rebase pulls in changes from recently merged PRs.

Copy link

@leszek-bq leszek-bq left a comment

Choose a reason for hiding this comment

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

LGTM

@danielfrankcom danielfrankcom merged commit 915cafd into version-0 Sep 26, 2025
11 checks passed
@danielfrankcom danielfrankcom deleted the dev/frankcom/server-side-cursors branch September 26, 2025 18:33
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.

3 participants