-
Notifications
You must be signed in to change notification settings - Fork 5.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
Shenli/set password #195
Shenli/set password #195
Conversation
@@ -203,6 +203,9 @@ func (s *testParserSuite) TestParser0(c *C) { | |||
// SET CHARACTER SET | |||
{"SET CHARACTER SET utf8mb4;", true}, | |||
{"SET CHARACTER SET 'utf8mb4';", true}, | |||
// Set password | |||
{"SET PASSWORD = 'password';", true}, | |||
{"SET PASSWORD FOR 'shenli'@'localhost' = 'password';", true}, |
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 root may be better than your name......
PTAL |
LGTM |
createDBSql string | ||
dropDBSql string | ||
useDBSql string | ||
createTableSql string |
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.
Replace Sql to Text would be 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.
They are all SQL statements.
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.
Yes, SQL is structured query language, it doesn't means text.
But we do mean text here.
PTAL |
LGTM |
* restore: add error field to `DownloadResponse` Signed-off-by: 5kbpers <tangminghua@pingcap.com>
pingcap#195) * hotfix the bug that -T can't work which is introduced from pingcap#194 * use sameStringArray
…mpling#194 (pingcap#195) * hotfix the bug that -T can't work which is introduced from pingcap/dumpling#194 * use sameStringArray
No description provided.