jayamanikharyono commented on code in PR #69382:
URL: https://github.com/apache/airflow/pull/69382#discussion_r3611716056
##########
providers/tableau/src/airflow/providers/tableau/operators/tableau.py:
##########
@@ -75,6 +82,11 @@ class TableauOperator(BaseOperator):
when ``exponential_backoff`` is enabled. ``None`` leaves the growth
uncapped.
:param incremental_refresh: Whether to perform an incremental refresh
instead of a full refresh.
Only applies to datasource and workbook refresh operations. Defaults
to False (full refresh).
+ :param skip_on_conflict: When ``True``, treat a Tableau ``409093 Resource
Conflict`` error
Review Comment:
Yes correct, applied both for Tableau Resource Refresh and Tableau Task Run.
I also agree that using an Airflow Pool or another queuing mechanism to
serialize refresh requests would be a good enhancement, but I think that's
outside the scope of this PR and could be considered as a future improvement.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
