vendor: update tipb, use shared bytes #2806

Merged
merged 1 commit into from Mar 10, 2017

Conversation

Projects
None yet
3 participants
@coocood
Member

coocood commented Mar 10, 2017

Reduce memory allocation.

Heap stats after scan 1 million record table:

before:

# Alloc = 1617988216
# TotalAlloc = 5759754344

after:

# Alloc = 1305848152
# TotalAlloc = 4735086792

The next step is to update kvproto to use shared bytes.

vendor: update tipb, use shared bytes
Reduce memory allocation.
@hanfei1991

This comment has been minimized.

Show comment
Hide comment
@hanfei1991

hanfei1991 Mar 10, 2017

Member

LGTM

Member

hanfei1991 commented Mar 10, 2017

LGTM

@tiancaiamao

This comment has been minimized.

Show comment
Hide comment
@tiancaiamao

tiancaiamao Mar 10, 2017

Contributor

LGTM

Contributor

tiancaiamao commented Mar 10, 2017

LGTM

@coocood coocood merged commit 27405e8 into master Mar 10, 2017

4 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details

@coocood coocood deleted the coocood/update-tipb branch Mar 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment