----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5347/ -----------------------------------------------------------
(Updated July 16, 2012, 9:31 a.m.) Review request for Flume. Changes ------- Mike's review comments are addressed. Thanks. Description ------- Redis supports pub/sub out-of-the-box. Flume based event streaming application can make use of Redis publisher sink to publish events to topic(s) in Redis and then subscriber(s) of topic/channel can receive events to do near real-time analytics. This addresses bug FLUME-1251. https://issues.apache.org/jira/browse/FLUME-1251 Diffs (updated) ----- trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkConfiguration.java 1361931 trunk/flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkType.java 1361931 trunk/flume-ng-dist/pom.xml 1361931 trunk/flume-ng-sinks/flume-ng-redis-sink/pom.xml PRE-CREATION trunk/flume-ng-sinks/flume-ng-redis-sink/src/main/java/org/apache/flume/sink/redis/RedisEventSerializer.java PRE-CREATION trunk/flume-ng-sinks/flume-ng-redis-sink/src/main/java/org/apache/flume/sink/redis/RedisSink.java PRE-CREATION trunk/flume-ng-sinks/flume-ng-redis-sink/src/main/java/org/apache/flume/sink/redis/SimpleRedisEventSerializer.java PRE-CREATION trunk/flume-ng-sinks/flume-ng-redis-sink/src/test/java/org/apache/flume/sink/redis/TestRedisSink.java PRE-CREATION trunk/flume-ng-sinks/pom.xml 1361931 trunk/pom.xml 1361931 Diff: https://reviews.apache.org/r/5347/diff/ Testing ------- Yes, tested in lab environment. Thanks, Mubarak Seyed