Skip to content

updating the code for _sql_accounts_base and sql_grants views to chec…#12

Open
ikewalker wants to merge 1 commit intomasterfrom
issue_6
Open

updating the code for _sql_accounts_base and sql_grants views to chec…#12
ikewalker wants to merge 1 commit intomasterfrom
issue_6

Conversation

@ikewalker
Copy link
Collaborator

…k whether the mysql.user.password column exists or not, and use the authentication_string column in its place if password does not exist

…k whether the mysql.user.password column exists or not, and use the authentication_string column in its place if password does not exist
@shlomi-noach
Copy link
Owner

Hi @ikewalker ,
a simpler solution is to just wrap the CREATE within a try-catch. This is what I do for the Percona views and for the TokuDB views.
See:

or see the complete common_schema....sql file to see how ant generates and wraps the try-catch.

The reason I like this way better is that I don't need to check for conditions. I just try it brute force, and if it works, it works. If not, it fails silently and continues deployment.

@ahathaway
Copy link

@ikewalker any progress on this?

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