fix(data-warehouse): Ensure last synced at is always updated

[Gilbert09]

Problem

• If a sync occurs and no data is returned (e.g. in an incremental sync), then the schemas last_synced_at time isn't updated, making it look like it wasn't actually ran

Changes

Ensure the datetime is always updated when we have a successful sync

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Updated end to end tests to include this

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ OperationalError: connection failed: connection to server at "172.20.49.244", port 6543 failed: server closed the c... posthog.warehouse.models.external_data_schema i... View Issue

_{Did you find this useful? React with a 👍 or 👎}