-
Notifications
You must be signed in to change notification settings - Fork 996
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
DQ base workflow #13454
DQ base workflow #13454
Conversation
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
# First, create the executable test suite if it does not exist yet | ||
# This could happen if the process is executed from YAML and not the UI | ||
if record.executable_test_suite: | ||
# We pass the test suite request to the sink | ||
return Either(right=record.executable_test_suite) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this be handle in source/test_suite.py
ln 172? If we return this in this step wouldn't we be just passing the testSuite to the sink?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes exactly that. In the source we return 2 possibilities:
- First, if needed, the executable test suite to be created. In that case, the processor does not need to do anything, just pass it to the sink to get it created
- Then, pick up the test cases (if any) and return them with the table. Now it's where the processor needs to get the cases from the CLI, execute them etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so in the case where we have a table without executable test suite, the source will first return the suite to be created, and then the rest of the logic will happen
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one comment. 😊
[open-metadata-airflow-apis] Kudos, SonarCloud Quality Gate passed! |
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
Describe your changes:
Fixes
I worked on ... because ...
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>