Skip to content

Shenli/fix jdbc auth#264

Merged
shenli merged 4 commits intomasterfrom
shenli/fix-jdbc-auth
Sep 24, 2015
Merged

Shenli/fix jdbc auth#264
shenli merged 4 commits intomasterfrom
shenli/fix-jdbc-auth

Conversation

@shenli
Copy link
Copy Markdown
Member

@shenli shenli commented Sep 24, 2015

Fix jdbc auth error

@coocood
Copy link
Copy Markdown
Member

coocood commented Sep 24, 2015

LGTM

Comment thread tidb-server/server/server.go Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Call Seed once is fine. I think you may call it when in initialization function of package .

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

PTAL

@siddontang
Copy link
Copy Markdown
Member

LGTM

shenli added a commit that referenced this pull request Sep 24, 2015
@shenli shenli merged commit ee03f7e into master Sep 24, 2015
@ngaut ngaut deleted the shenli/fix-jdbc-auth branch October 29, 2015 10:57
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.

4 participants