Fix SAMPLES_QUERY deterministic ordering per CR comment

devin-ai-integration[bot] · arbiv · devin-ai-integration[bot] · commit c56e9162fea4 · 2025-07-29T08:00:02.000Z
- Add 'id desc' as secondary sort key to ensure deterministic results
- Addresses CodeRabbit comment about non-deterministic ordering when timestamps tie

Co-Authored-By: Yosef Arbiv &lt;yosef.arbiv@gmail.com&gt;
diff --git a/integration_tests/tests/test_sampling_pii.py b/integration_tests/tests/test_sampling_pii.py
@@ -11,7 +11,7 @@
         select id
         from {{{{ ref("elementary_test_results") }}}}
         where lower(table_name) = lower('{test_id}')
-        order by created_at desc
+        order by created_at desc, id desc
         limit 1
     )
 

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`select id`
`12`	`12`	`from {{{{ ref("elementary_test_results") }}}}`
`13`	`13`	`where lower(table_name) = lower('{test_id}')`
`14`		`- order by created_at desc`
	`14`	`+ order by created_at desc, id desc`
`15`	`15`	`limit 1`
`16`	`16`	`)`
`17`	`17`