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

[PLUGIN-1773] Add a check for error 71005 #6

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

psainics
Copy link

@psainics psainics commented Mar 27, 2024

Add a check for soap error 71005

Jira : PLUGIN-1773

Description

Add a check for error code 71005 (Violation of PRIMARY KEY constraint)

@psainics psainics added the bug Something isn't working label Mar 27, 2024
@psainics psainics requested a review from vikasrathee-cs March 27, 2024 07:54
@psainics psainics self-assigned this Mar 27, 2024
@psainics psainics changed the title Add a check for error 71005 [PLUGIN-1773] Add a check for error 71005 Mar 27, 2024
@@ -205,7 +205,8 @@ public List<ETResult<ETDataExtensionRow>> upsert(List<ETDataExtensionRow> rows)
List<ETDataExtensionRow> toUpdate = new ArrayList<>();
for (ETResult<ETDataExtensionRow> row : inserts.getResults()) {
// super hacky to check the error message... but there is no better way
if (row.getStatus() == ETResult.Status.ERROR && row.getErrorCode() == 2 && row.getErrorMessage() != null &&
if (row.getStatus() == ETResult.Status.ERROR && row.getErrorCode() != null &&
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add a test case and add a comment giving more details about 71005

@psainics psainics force-pushed the patch/err_71005 branch 3 times, most recently from dced125 to e5dcb4c Compare April 4, 2024 02:09
@psainics psainics merged commit da72150 into develop Apr 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants