Skip to content

Commit 6014bac

Browse files
Resolve merge conflicts - keep column exclusion implementation
- Keep unified get_columns_to_exclude_from_sampling approach - Maintain updated tests that expect column exclusion behavior - Remove early return logic that skipped sampling entirely - Address GitHub feedback to exclude columns, not skip sampling Co-Authored-By: Yosef Arbiv <[email protected]>
2 parents 9fa0eb1 + 3ba7a44 commit 6014bac

File tree

1 file changed

+1
-0
lines changed
  • macros/edr/materializations/test

1 file changed

+1
-0
lines changed

macros/edr/materializations/test/test.sql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -180,6 +180,7 @@
180180
{% do return(false) %}
181181
{% endmacro %}
182182

183+
183184
{% macro cache_elementary_test_results_rows(elementary_test_results_rows) %}
184185
{% do elementary.get_cache("elementary_test_results").update({model.unique_id: elementary_test_results_rows}) %}
185186
{% endmacro %}

0 commit comments

Comments
 (0)