Skip to content

Commit 1befbaf

Browse files
committed
Split checks into stages
1 parent 22ca2de commit 1befbaf

File tree

114 files changed

+590
-412
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

114 files changed

+590
-412
lines changed

CHANGELOG.md

Lines changed: 4 additions & 1 deletion

flowdb/testdata/bin/run_qa_checks.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,7 @@ def get_available_dates(engine: Engine):
107107

108108
templates = (
109109
QaTemplate(
110-
display_name=(
111-
".".join((Path(t).stem, Path(t).parent.stem))
112-
if Path(t).parent != Path(".")
113-
else Path(t).stem
114-
),
110+
display_name=Path(t).stem,
115111
template=env.get_template(t),
116112
event_type=Path(t).parent.stem if Path(t).parent != Path(".") else "any",
117113
)

flowdb_synthetic_data.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ ADD --chown=postgres flowdb/testdata/test_data/sql/admin*.sql /docker-entrypoint
5050
ADD --chown=postgres flowdb/testdata/synthetic_data/data/NPL_admbnda_adm3_Districts_simplified.geojson /opt/synthetic_data/
5151
# Copy QA templates from flowetl
5252

53-
COPY --chown=postgres flowetl/flowetl/flowetl/qa_checks/qa_checks /docker-entrypoint-initdb.d/qa_checks
53+
COPY --chown=postgres flowetl/flowetl/flowetl/qa_checks/final /docker-entrypoint-initdb.d/qa_checks
5454

5555
# Need to make postgres is owner of any subdirectrories
5656
RUN mkdir docker-entrypoint-initdb.d/sql/syntheticdata/sql && chown -R postgres /docker-entrypoint-initdb.d

flowdb_synthetic_data.Dockerfile.dockerignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,4 @@
1919
!flowdb/testdata/bin/9910_run_synthetic_dfs_data_generation_script.sh
2020
!flowdb/testdata/bin/run_qa_checks.py
2121
!flowdb/testdata/test_data/py/
22-
!flowetl/flowetl/flowetl/qa_checks/qa_checks
22+
!flowetl/flowetl/flowetl/qa_checks/

flowdb_testdata.Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ ADD --chown=postgres flowdb/testdata/test_data/sql/* /docker-entrypoint-initdb.d
4343
ADD --chown=postgres flowdb/testdata/test_data/py/* /docker-entrypoint-initdb.d/py/testdata/
4444
ADD --chown=postgres flowdb/testdata/test_data/data/ /docker-entrypoint-initdb.d/data/
4545
COPY --chown=postgres flowdb/testdata/test_data/data/*.csv /docker-entrypoint-initdb.d/data/csv/
46-
COPY --chown=postgres flowetl/flowetl/flowetl/qa_checks/qa_checks /docker-entrypoint-initdb.d/qa_checks
46+
COPY --chown=postgres flowetl/flowetl/flowetl/qa_checks/final /docker-entrypoint-initdb.d/qa_checks
4747
# Need to make postgres owner of any subdirectories
4848
RUN chown -R postgres /docker-entrypoint-initdb.d
4949
# Explicitly set number of days of dfs data to match test data

flowdb_testdata.Dockerfile.dockerignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,5 @@
1919
!flowdb/testdata/test_data/sql/
2020
!flowdb/testdata/test_data/py/
2121
!flowdb/testdata/test_data/data/
22-
!flowetl/flowetl/flowetl/qa_checks/qa_checks
22+
!flowetl/flowetl/flowetl/qa_checks/
2323

flowetl/flowetl/MANIFEST.in

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,4 @@ include versioneer.py
22
include flowetl/_version.py
33
include flowetl/qa_checks/*/*
44
include flowetl/qa_checks/*/*/*
5+
include flowetl/qa_checks/*/*/*/*

flowetl/flowetl/flowetl/operators/qa_check_operator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
QACheckOperator = wrapped_sql_operator(
88
sql="""INSERT INTO etl.post_etl_queries(cdr_date, cdr_type, type_of_query_or_check, outcome, timestamp)
99
VALUES
10-
(date '{{{{ ds }}}}', '{{{{ params.cdr_type }}}}', '{{{{ task.task_id }}}}', ({sql}), NOW())
10+
(date '{{{{ ds }}}}', '{{{{ params.cdr_type }}}}', '{{{{ task.task_id.split(".")[0] }}}}', ({sql}), NOW())
1111
""",
1212
class_name="QACheckOperator",
1313
)

flowetl/flowetl/flowetl/qa_checks/extract/.gitkeep

Whitespace-only changes.
File renamed without changes.

0 commit comments

Comments
 (0)