Skip Tableau refresh/task run on 409093 resource conflict#69382
Open
jayamanikharyono wants to merge 1 commit into
Open
Skip Tableau refresh/task run on 409093 resource conflict#69382jayamanikharyono wants to merge 1 commit into
jayamanikharyono wants to merge 1 commit into
Conversation
jayamanikharyono
force-pushed
the
tableau-refresh-skip-on-conflict
branch
from
July 6, 2026 13:56
5b82ba5 to
9da6df9
Compare
jayamanikharyono
marked this pull request as draft
July 6, 2026 13:57
jayamanikharyono
marked this pull request as ready for review
July 6, 2026 14:03
jayamanikharyono
force-pushed
the
tableau-refresh-skip-on-conflict
branch
3 times, most recently
from
July 7, 2026 13:53
5a93f41 to
973c4d6
Compare
potiuk
added
the
ready for maintainer review
jayamanikharyono
force-pushed
the
tableau-refresh-skip-on-conflict
branch
6 times, most recently
from
July 18, 2026 13:18
933aae9 to
3eaf940
Compare
jroachgolf84
suggested changes
Jul 19, 2026
jroachgolf84
left a comment
jroachgolf84
left a comment
Collaborator
There was a problem hiding this comment.
Good start!
Are there tests for when incremental_refresh == True? Since there are changes there, I'd recommend adding those tests.
jayamanikharyono
force-pushed
the
tableau-refresh-skip-on-conflict
branch
from
July 20, 2026 04:48
3eaf940 to
1f57d4e
Compare
Add `skip_on_conflict` support and improve conflict messages.
jayamanikharyono
force-pushed
the
tableau-refresh-skip-on-conflict
branch
from
July 24, 2026 03:25
1f57d4e to
c8c1fd6
Compare
jroachgolf84
approved these changes
Jul 24, 2026
jroachgolf84
left a comment
jroachgolf84
left a comment
Collaborator
There was a problem hiding this comment.
This LGTM - were you able to test this on an actual Tableau instance?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a
skip_on_conflictoption toTableauOperatorto allow Tableau409093 Resource Conflicterrors to be treated as skipped instead of failed.A
409093error indicates that a refresh or task run for the same resource has already been queued or is currently running. In these cases, failing the task may not be desirable, especially for overlapping schedules or retries where the requested operation is already in progress.When
skip_on_conflict=True, the operator raisesAirflowSkipExceptionfor409093responses. Otherwise, the existing behavior is preserved. The option defaults toFalseto maintain backward compatibility.This behavior applies to both supported Tableau operations:
method="refresh"for datasources and workbooksmethod="run"for tasksSupporting docs :
Was generative AI tooling used to co-author this PR?