-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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(ingest/sql-common): Fix profile_table_level_only #8331
fix(ingest/sql-common): Fix profile_table_level_only #8331
Conversation
@@ -351,65 +351,7 @@ | |||
}, | |||
"rowCount": 2, | |||
"columnCount": 9, | |||
"fieldProfiles": [ |
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.
just confirming that this change is actually correct
kinda hard to understand the correspondence between the yml files and the golden files - they really should've been named better
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.
Yup, this is what no fields profiled looks like
This fixes the bug but I do not know why we're supporting profiling some table level values when
profile_table_level_only
is provided. Also makes it so we don't send field profiles if we're not ingesting any columns.Checklist