-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Add current_user
alias
#61770
Add current_user
alias
#61770
Conversation
This is an automated comment for commit b3692fe with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
Add a test. |
Don't forget to add docs to https://clickhouse.com/docs/en/sql-reference/functions/other-functions#currentUser. |
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.
Approving the docs section of this PR. I'll leave it to the engineers to approve the actual code.
The current test turned out to be flaky. As the previous check of alias here was simple, I will make similar checks instead and will investigate the root cause of this strange behavior in one of the next PRs. |
Added
current_user
as a compatibility alias for MySQL.Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added
current_user
function as a compatibility alias for MySQL.Documentation entry for user-facing changes
Modify your CI run:
NOTE: If your merge the PR with modified CI you MUST KNOW what you are doing
NOTE: Set desired options before CI starts or re-push after updates
Run only:
CI options:
Only specified batches in multi-batch jobs: