Skip to content

Commit

Permalink
Minor
Browse files Browse the repository at this point in the history
  • Loading branch information
DVAlexHiggs committed Sep 14, 2023
1 parent 0896d72 commit 0552e15
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/best_practises/loading.md
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ This audit column records when the record was first loaded into the database.
The Load Date/Timestamp should be the same for every record loaded in a batch, for every table loaded - this means that if 5 tables are being loaded in parallel,
they should all have the same value for ldts. It is not correct to have the time that the model (sql file) itself gets executed as it will cause problems for audit.

THe above information explains why using something like `CURRENT_TIMESTAMP` - though seemingly a good idea at first - does not make sense for the LDTS value.
The above information explains why using something like `CURRENT_TIMESTAMP` - though seemingly a good idea at first - does not make sense for the LDTS value.

Ideally, you should set it to a record generated from your ingestion tool of choice. For example when using Fivetran, it is often sensible to use `_FIVETRAN_SYNCED`.

Expand Down

0 comments on commit 0552e15

Please sign in to comment.