Skip to content

add column user_max to history table, as describled in https://www.percona.c…#142

Open
hankya wants to merge 1 commit into
box:masterfrom
hankya:master
Open

add column user_max to history table, as describled in https://www.percona.c…#142
hankya wants to merge 1 commit into
box:masterfrom
hankya:master

Conversation

@hankya

@hankya hankya commented Oct 10, 2015

Copy link
Copy Markdown

…om/doc/percona-toolkit/2.0/pt-query-digest.html#cmdoption-pt-query-digest--review-history, this will cause pt-query-digest tool to store user info to this column.

This solves issue #138

@boxcla

boxcla commented Oct 10, 2015

Copy link
Copy Markdown

Hi @hankya, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla

Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

@hankya

hankya commented Oct 12, 2015

Copy link
Copy Markdown
Author

CLA signed.

@boxcla

boxcla commented Oct 12, 2015

Copy link
Copy Markdown

I couldn't find a signed CLA for @hankya. Can you please try again?

@hankya

hankya commented Oct 19, 2015

Copy link
Copy Markdown
Author

Signed again, could you please verify?

@akuzminsky

Copy link
Copy Markdown
Contributor

@hankya boxcla is a bot. I guess it's waiting for a "CLA signed." comment.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


yangrun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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