Skip to content

Performance tuning: 20% improvements for simple put operations#24

Open
yjh0502 wants to merge 1 commit intohoisie:masterfrom
yjh0502:perf-patch
Open

Performance tuning: 20% improvements for simple put operations#24
yjh0502 wants to merge 1 commit intohoisie:masterfrom
yjh0502:perf-patch

Conversation

@yjh0502
Copy link
Copy Markdown

@yjh0502 yjh0502 commented Jul 10, 2013

I tuned some function in critical path.

  • Change fmt.Sprintf to bytes.Buffer
  • Change string operations to byte operations
  • Change default buffer size to handle small-sized request better

These update shows 20% improvement for simple set operations. See following link for detail: https://docs.google.com/spreadsheet/ccc?key=0AjFZD5XgU6lFdEhPS21Pd0lpYjFKcGJLalRCdG9UZ3c#gid=0

@codeskyblue
Copy link
Copy Markdown

great

1 similar comment
@sunminghong
Copy link
Copy Markdown

great

@hoisie
Copy link
Copy Markdown
Owner

hoisie commented Mar 19, 2014

This looks great, but it needs a merge with master - are you available to do it?

@yjh0502
Copy link
Copy Markdown
Author

yjh0502 commented Mar 20, 2014

@hoisie I'll merge the patch with master, but not right now because I don't have a spare time right now. It may takes some time, maybe several days. I'll update this PR after merge :)

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