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
Remove filter profile info #1481
Conversation
…raph into remove-filter-profile-info
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 think the name of this PR is a bit misleading since you aren't actually remove filter profile info but rather changing the output of filter explain.
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.
Few comments
SonarCloud Quality Gate failed. 0 Bugs No Coverage information Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Filter profile info was not updating during rewriting of queries.
[master < Task] PR
To keep docs changelog up to date, one more thing to do:
@vpavicic : Changelog message: Filter information message fixed in EXPLAIN and PROFILE queries to display what's exactly filtering in the filter expression.
@vpavicic I think I'll need to look about what currently our profile and explain information provides in the docs so we can update it