Skip to content

fix stats.Counters.String method to generate valid JSON for map keys#3005

Merged
sougou merged 1 commit intovitessio:masterfrom
nerdatmath:master
Jul 24, 2017
Merged

fix stats.Counters.String method to generate valid JSON for map keys#3005
sougou merged 1 commit intovitessio:masterfrom
nerdatmath:master

Conversation

@nerdatmath
Copy link
Copy Markdown
Contributor

No description provided.

@sougou
Copy link
Copy Markdown
Contributor

sougou commented Jul 23, 2017

@bbeaudreault can you verify this will work for you?

@bbeaudreault
Copy link
Copy Markdown
Contributor

Cool, I did not know about %q. I'll roll this out internally to verify, but a quick go playground example looked good.

@bbeaudreault
Copy link
Copy Markdown
Contributor

bbeaudreault commented Jul 24, 2017

Tested internally and LGTM. Thanks!

@sougou
Copy link
Copy Markdown
Contributor

sougou commented Jul 24, 2017

LGTM

Approved with PullApprove

@sougou sougou merged commit da70349 into vitessio:master Jul 24, 2017
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