Skip to content

feat(mysql)!: support user() for mysql#7002

Closed
AbhishekASLK wants to merge 1 commit intotobymao:mainfrom
AbhishekASLK:abhishekaslk/support-user
Closed

feat(mysql)!: support user() for mysql#7002
AbhishekASLK wants to merge 1 commit intotobymao:mainfrom
AbhishekASLK:abhishekaslk/support-user

Conversation

@AbhishekASLK
Copy link
Contributor

✳️ This PR map exp.CurrentUser to USER for MySQL

select current_user(), user();
+----------------------------+--------------------------------------+
| current_user()             | user()                               |
+----------------------------+--------------------------------------+
| user_44ctrvzeh_44cvj2esn@% | user_44ctrvzeh_44cvj2esn@172.17.0.10 |
+----------------------------+--------------------------------------+

Copy link
Collaborator

@VaggelisD VaggelisD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not correct despite the common values, USER and CURRENT_USER have different semantics

@AbhishekASLK
Copy link
Contributor Author

This is not correct despite the common values, USER and CURRENT_USER have different semantics

Thanks for pointing this out.

@georgesittas
Copy link
Collaborator

@AbhishekASLK any updates here?

@AbhishekASLK
Copy link
Contributor Author

@AbhishekASLK any updates here?

Hi @georgesittas

As both have different meaning for MySQL, then we can close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants