Skip to content

Commit

Permalink
fix: Use time instead of vector_timestamp for sorting
Browse files Browse the repository at this point in the history
The `vector_timestamp` column doesn't have any semantic meaning for the tracking
logs. This removes use of that column (which was renamed to `timestamp`) in favor of
`time` which is the actual event time.
  • Loading branch information
blarghmatey committed Oct 2, 2023
1 parent 083cdae commit fc3173e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ with source as (
*
, row_number() over (
partition by username, context, event_source, event_type, event, "time" --noqa
order by _airbyte_emitted_at desc, _ab_source_file_last_modified desc, vector_timestamp desc
order by _airbyte_emitted_at desc, _ab_source_file_last_modified desc, "time" desc
) as row_num
from source
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ with source as (
*
, row_number() over (
partition by username, context, event_source, event_type, event, "time" --noqa
order by _airbyte_emitted_at desc, _ab_source_file_last_modified desc, vector_timestamp desc
order by _airbyte_emitted_at desc, _ab_source_file_last_modified desc, "time" desc
) as row_num
from source
)
Expand Down

0 comments on commit fc3173e

Please sign in to comment.