-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26012/
-----------------------------------------------------------

(Updated Sept. 25, 2014, 3:12 p.m.)


Review request for Aurora, David McLaughlin, Joe Smith, and Bill Farner.


Changes
-------

Update Testing Done to reflect current code


Bugs: AURORA-722
    https://issues.apache.org/jira/browse/AURORA-722


Repository: aurora


Description
-------

Use BufferedOutputStream for deflater


Diffs (updated)
-----

  src/main/java/org/apache/aurora/codec/ThriftBinaryCodec.java 
45cf7ecf52582ba3fd7bbc8b4a981e396793da4b 

Diff: https://reviews.apache.org/r/26012/diff/


Testing (updated)
-------

./gradlew build

Microbenchmark with a scheduler backup from my production environment.

Before:
## Deflater.DEFAULT_COMPRESSION, (No BufferedOutputStream), 
DeflaterOutputStream(
deflate: 16664ms
total:17219ms
compression ratio: 647270930/92690409 = 698.31%

After:
# Deflater.DEFAULT_COMPRESSION, BufferedOutputStream(256KiB), 
DeflaterOutputStream(256KiB)
deflate: 11807ms (4857ms speedup)
total: 12379ms (4840ms speedup)
compression ratio: 647270930/92690409 = 698.31% (0% improvement)


Thanks,

Kevin Sweeney

Reply via email to