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

API Changes to FivetranSensor for supporting backfills #58

Merged
merged 10 commits into from
Nov 6, 2023
Prev Previous commit
Next Next commit
Merge branch 'main' into api-changes
  • Loading branch information
dwreeves committed Nov 2, 2023
commit c626478acc432fcbbf94d7e6a82a8f8a6e35dd4d
8 changes: 3 additions & 5 deletions fivetran_provider_async/hooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -164,9 +164,9 @@ def _do_api_call(
if not _retryable_error(e):
# In this case, the user probably made a mistake.
# Don't retry.
assert e.response is not None # mypy
assert e.response is not None
raise AirflowException(
f"Response: {e.response.content!r}, " f"Status Code: {e.response.status_code}"
f"Response: {e.response.content.decode()}, " f"Status Code: {e.response.status_code}"
)

self._log_request_error(attempt_num, str(e))
Expand Down Expand Up @@ -476,9 +476,7 @@ def _determine_if_synced_from_connector_details(
# Check if sync started by FivetranOperator has finished
# indicated by new 'succeeded_at' timestamp
if succeeded_at > completed_after_time:
self.log.info(
"Connector %s: succeeded_at: %s, connector_id", connector_id, succeeded_at.to_iso8601_string()
)
self.log.info("Connector %s: succeeded_at: %s", connector_id, succeeded_at.to_iso8601_string())
return True

# if sync in rescheduled start, wait for time recommended by Fivetran
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.