----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31836/ -----------------------------------------------------------
(Updated March 11, 2015, 3:49 p.m.) Review request for helix. Changes ------- incorporated the feedback, also added the code in external view compute stage to copy the enableCompression from Idealstate to ExternalView Repository: helix-git Description ------- commit 1ef44c2e9a132df3513a51e3a8dac658236a2263 Author: Kishore Gopalakrishna <[email protected]> Date: Sun Mar 8 16:40:29 2015 -0700 [HELIX-573] Add support to automatically compress/uncompress data in Zookeeper :100644 100644 4419fdd... 1f34529... M helix-core/src/main/java/org/apache/helix/manager/zk/ZNRecordSerializer.java :100644 100644 2d7cb3c... 26d7e2b... M helix-core/src/main/java/org/apache/helix/manager/zk/ZNRecordStreamingSerializer.java :000000 100644 0000000... 90c1e8e... A helix-core/src/test/java/org/apache/helix/manager/zk/TestZNRecordSerializer.java :100644 100644 e4b0b25... 95064f8... M helix-core/src/test/java/org/apache/helix/manager/zk/TestZNRecordStreamingSerializer.java :100755 100755 e869f25... 99ef81c... M hpost-review.sh Diffs (updated) ----- helix-core/src/main/java/org/apache/helix/controller/stages/ExternalViewComputeStage.java 903840c helix-core/src/main/java/org/apache/helix/manager/zk/ZNRecordSerializer.java 4419fdd helix-core/src/main/java/org/apache/helix/manager/zk/ZNRecordStreamingSerializer.java 2d7cb3c helix-core/src/main/java/org/apache/helix/util/GZipCompressionUtil.java PRE-CREATION helix-core/src/test/java/org/apache/helix/integration/TestEnableCompression.java PRE-CREATION helix-core/src/test/java/org/apache/helix/manager/zk/TestZNRecordSerializer.java PRE-CREATION helix-core/src/test/java/org/apache/helix/manager/zk/TestZNRecordStreamingSerializer.java e4b0b25 Diff: https://reviews.apache.org/r/31836/diff/ Testing ------- Added basic test for compress/uncompress Thanks, Kishore Gopalakrishna
