Skip to content

dbeaver/pro#8899 disable autocommit toggle for db2i#4311

Open
Nexus6v2 wants to merge 3 commits intodevelfrom
dbeaver/pro#8899-db2i-autocommits
Open

dbeaver/pro#8899 disable autocommit toggle for db2i#4311
Nexus6v2 wants to merge 3 commits intodevelfrom
dbeaver/pro#8899-db2i-autocommits

Conversation

@Nexus6v2
Copy link
Copy Markdown
Contributor

Closes dbeaver/pro#8899

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 29, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Disables explicit auto-commit toggling during result-set batch updates for data sources that implement DBCNativeBatchAutoCommit (intended to address DB2i behavior per dbeaver/pro#8899).

Changes:

  • Adds a skipAutoCommitToggle guard to avoid calling setAutoCommit(false) for certain data sources.
  • Extends the same guard to suppress explicit commit/rollback calls during the batch update flow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Nexus6v2 Nexus6v2 requested a review from serge-rider April 29, 2026 14:57
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