Skip to content

[PLUGIN-1949] Add Oracle UPSERT#645

Open
psainics wants to merge 2 commits intodata-integrations:developfrom
cloudsufi:orc-upsert
Open

[PLUGIN-1949] Add Oracle UPSERT#645
psainics wants to merge 2 commits intodata-integrations:developfrom
cloudsufi:orc-upsert

Conversation

@psainics
Copy link
Contributor

@psainics psainics commented Jan 30, 2026

Add Oracle UPSERT

Jira : PLUGIN-1945

This pull request introduces support for upsert operations in the Oracle batch sink plugin.

  • Added a new class OracleETLDBOutputFormat that extends ETLDBOutputFormat and implements the constructUpsertQuery method, generating an Oracle-compatible MERGE-based upsert SQL statement.
  • Updated OracleSink to use the new OracleETLDBOutputFormat by adding an output context with SinkOutputFormatProvider.

UI Update

image

@psainics psainics requested a review from prince-cs January 30, 2026 03:09
@psainics psainics self-assigned this Jan 30, 2026
@psainics psainics added the build label Jan 30, 2026
Copy link
Contributor

@prince-cs prince-cs left a comment

Choose a reason for hiding this comment

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

LGTM

@psainics psainics force-pushed the orc-upsert branch 2 times, most recently from 8cacac6 to 5ea49a5 Compare February 2, 2026 09:29
@psainics psainics requested a review from prince-cs February 19, 2026 04:35
@psainics psainics marked this pull request as ready for review February 19, 2026 04:35
Copy link
Contributor

@itsankit-google itsankit-google left a comment

Choose a reason for hiding this comment

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

Please add e2e tests for upsert / update cases.

@psainics psainics changed the title Add Oracle UPSERT [PLUGIN-1949] Add Oracle UPSERT Mar 23, 2026
| upsert |
| update |

@BQ_SOURCE_TEST @ORACLE_TEST_TABLE @Oracle_Required
Copy link
Contributor

Choose a reason for hiding this comment

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

This test does not actually test upsert / update.

The @ORACLE_TEST_TABLE tag just creates the source table :

public static void createOracleTargetTestTable() throws SQLException, ClassNotFoundException {

But does not insert any data, then in this case upsert / update will work as insert.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants