Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ele 2601 override primary_test_model_id #671

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

NoyaArie
Copy link
Contributor

No description provided.

Copy link

linear bot commented Feb 28, 2024

Copy link
Contributor

👋 @NoyaArie
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in the elementary repository.

Copy link
Contributor

@erikzaadi erikzaadi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NoyaArie NoyaArie merged commit e61292b into master Feb 29, 2024
10 of 11 checks passed
@NoyaArie NoyaArie deleted the ele-2601-override-primary-model-singular-tests branch February 29, 2024 09:43
konstantin-baidin-y42 added a commit to goes-funky/dbt-data-reliability that referenced this pull request Apr 9, 2024
* support count_true and count_false for boolean columns in BigQuery

* support count_true and count_false for boolean columns in BigQuery

* fix: change generate profile args macro name for Athena

* Add unique_combination_of_columns to common_tests_configs_mapping

* missing comma

* fix drop_failure_percent_threshold failing a non anomalous test

* Support all anomaly vars on all configuration levels

* ELE-2470 temp tables are not being deleted

* Add empty line at the end of a filre

* fix typo

* Removed default detection/training_period

* Make sure we delete temp tables last

* removed unused import

* Not collecting metrics by default.

* release 0.14.0

* fix bug when no temp tables xist

* Update macros/edr/tests/test_utils/clean_elementary_test_tables.sql

Co-authored-by: IDoneShaveIt <48473443+IDoneShaveIt@users.noreply.github.com>

* 1. add tags to all elementary monitors 2. only run tests if elementary is enabled

* rename tag

* Create clean_dbt_columns_temp_tables macro

* Add empty line at the end of a file

* clean logs

* add arg chunk_size for all insert_rows() (elementary-data#669)

* release 0.14.1

* override primary_test_model_id (elementary-data#671)

* added ignore_small_changes to freshness and event_freshness

* improvement: bigquery specific for query_table_metrics (elementary-data#674)

* improvement: bigquery specific for query_table_metrics

Using information schema to get row count is much more performant than doing a full table scan

* use TABLE_STORAGE and add database & schema

* add empty case

* add set

* Add index on created_at test_result_rows and remove backfill post hook

* Ele 2606 package version with caching and extra logs (elementary-data#673)

* artifacts: use cache also for model post-hook

* add performance logs to artifacts logic

* duration monitoring - bugfix - handle the case the duration stack is not initialized

* Change the aggregate of failed_row_count_calc to count(*)

* Readme updates (elementary-data#684)

* changes to readme

* changes to readme

* changes

* changes

* image url

* image url

* changes

* formating

* formating

* changes

* link

* pre commit

* improve flattening performance for dbt_columns (elementary-data#681)

* improve flattening performance for dbt_columns

* removed unused const

* black

* Add get_requires_permissions and validate_required_permissions macros

* Improved messages

* Fixed default__get_required_permissions + add target.database to get_relevant_databases

---------

Co-authored-by: suelai <suela.isaj@gmail.com>
Co-authored-by: Roman Korsun <romakorsun2000@gmail.com>
Co-authored-by: Yasuhisa Yoshida <syou6162@gmail.com>
Co-authored-by: Ofek Weiss <55920061+ofek1weiss@users.noreply.github.com>
Co-authored-by: Ofek Weiss <ofek1weiss@gmail.com>
Co-authored-by: IDoneShaveIt <idanshavit31@gmail.com>
Co-authored-by: IDoneShaveIt <48473443+IDoneShaveIt@users.noreply.github.com>
Co-authored-by: Elon Gliksberg <elongliks@gmail.com>
Co-authored-by: GitHub Actions <noreply@github.com>
Co-authored-by: Ella Katz <ella@elementary-data.com>
Co-authored-by: J.C <zhang@sansan.com>
Co-authored-by: Noy Arie <noyarie1992@gmail.com>
Co-authored-by: Chris Dong <86695140+dongchris@users.noreply.github.com>
Co-authored-by: noakurman <kurman.noa@gmail.com>
Co-authored-by: Itamar Hartstein <haritamar@gmail.com>
Co-authored-by: Maayan Salom <maayansalom@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants