[GitHub] [geode] pivotal-jbarrett commented on issue #2970: GEODE-6166: Improves put performance
@upthewaterspout it does change the on wire and all backwards compatibility tests pass. Old clients send the serialized object an the server detects this because the part is and object. New clients send just a byte and the server detects this because the part is not an object. Saves ~63 bytes per put message. [ Full content available at: https://github.com/apache/geode/pull/2970 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode] pivotal-jbarrett commented on issue #2970: GEODE-6166: Improves put performance
@upthewaterspout it does change the on wire and all backwards compatibility tests pass. Old clients send the serialized object and the server detects this because the part is and object. New clients send just a byte and the server detects this because the part is not an object. Saves ~63 bytes per put message. [ Full content available at: https://github.com/apache/geode/pull/2970 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode] pivotal-jbarrett commented on issue #2970: GEODE-6166: Improves put performance
Opps, basic math error. Corrected numbers are: When benchmarked with a simple rapid put operation the changes between develop and this PR resulted in about ~25% increase in puts/second. Comparing the profiles between develop and the PR shows a ~42% reduction in allocated transient objects. Still great numbers! [ Full content available at: https://github.com/apache/geode/pull/2970 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org
[GitHub] [geode] pivotal-jbarrett commented on issue #2970: GEODE-6166: Improves put performance
When benchmarked with a simple rapid put operation the changes between develop and this PR resulted in about ~20% increase in puts/second. Comparing the profiles between develop and the PR shows a ~72% reduction in allocated transient objects. [ Full content available at: https://github.com/apache/geode/pull/2970 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org