NIFI-8142 Add "on conflict do nothing" feature to PutDatabaseRecord#4778
NIFI-8142 Add "on conflict do nothing" feature to PutDatabaseRecord#4778Snorlaxa wants to merge 1 commit intoapache:mainfrom
Conversation
|
This PR needs some reorganization, maybe save off the commit(s) where you changed code then reset your branch to match the latest |
|
+1 LGTM, ran contrib-check and tried on MySQL, PostgreSQL, and Oracle, all behavior was as expected. However the commit doesn't have the Jira case or description so I will change that before merging, as well as fixing a Checkstyle error. In the future please ensure you run a Maven build (at least for the affected modules) with the Thanks for the new feature! Merging to main |
NIFI-8142: Fix Checkstyle error Signed-off-by: Matthew Burgess <mattyb149@apache.org> This closes apache#4778
NIFI-8142: Fix Checkstyle error Signed-off-by: Matthew Burgess <mattyb149@apache.org> This closes apache#4778
I believe PutDatabaseRecord processor need a feature that we can ignore the record when it already exists in database.
In this implement I add a new Statement Type "INSERT_IGNORE", inspired by @markap14 .
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:
Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
Has your PR been rebased against the latest commit within the target branch (typically
main)?Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not
squashor use--forcewhen pushing to allow for clean monitoring of changes.For code changes:
mvn -Pcontrib-check clean installat the rootnififolder?LICENSEfile, including the mainLICENSEfile undernifi-assembly?NOTICEfile, including the mainNOTICEfile found undernifi-assembly?.displayNamein addition to .name (programmatic access) for each of the new properties?For documentation related changes:
Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.