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
Fix #2897 - Profiler CLI #2967
Fix #2897 - Profiler CLI #2967
Conversation
# Conflicts: # ingestion/src/metadata/orm_profiler/metrics/composed/duplicate_count.py # ingestion/src/metadata/orm_profiler/metrics/composed/null_ratio.py # ingestion/src/metadata/orm_profiler/metrics/registry.py # ingestion/src/metadata/orm_profiler/profiles/core.py # ingestion/tests/unit/profiler/test_metrics.py
[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed! |
config = FileSinkConfig.parse_obj(config_dict) | ||
return cls(ctx, config) | ||
|
||
def write_record(self, record: ProfileAndTests) -> None: |
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.
jinja template could have helped you here :)
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.
yeah.. not the greatest solution, just needed something quick for the demo 😅
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.
I'll get this fixed after we apply the new test result classes 🙏
Describe your changes :
This PR fixes #2897
We have added the
profile
command in the CLI, added some fixes from testing the solution against MSSQL and created a FileSink object to play around the solution E2E.Thanks!
Type of change :
Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.
Checklist:
Reviewers