Skip to content

Commit

Permalink
Update website/docs/reference/resource-configs/event-time.md
Browse files Browse the repository at this point in the history
Co-authored-by: Joel Labes <[email protected]>
  • Loading branch information
mirnawong1 and joellabes authored Jan 13, 2025
1 parent 9764ae0 commit 11b9f76
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion website/docs/reference/resource-configs/event-time.md
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ You can configure `event_time` for a [model](/docs/build/models), [seed](/docs/b

### Best practices

Set the `event_time` to the name of the field that represents the actual timestamp of the event (like `account_created_at`). The timestamp of the event should represent "at what time did the row occur" rather than an event ingestion date.
Set the `event_time` to the name of the field that represents the actual timestamp of the event (like `account_created_at`). The timestamp of the event should represent "at what time did the row occur" rather than an event ingestion date. Marking a column as the `event_time` when it isn't diverges from the semantic meaning of the column which may result in user confusion when other tools make use of the metadata.

However, if an ingestion date (like `loaded_at`, `ingested_at`, or `last_updated_at`) are the only timestamps you use, you can set `event_time` to these fields. Here are some considerations to keep in mind if you do this:

Expand Down

0 comments on commit 11b9f76

Please sign in to comment.