----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41529/ -----------------------------------------------------------
(Updated Dec. 28, 2015, 6:26 p.m.) Review request for Flume and Ted Malaska. Changes ------- Corrected logic in HBaseSink.java Repository: flume-git Description ------- FLUME-2858, Added better exception message for malformed zookeeperQuorum. Added default port 2181 if the zookeeperQuorum port is not given. Diffs ----- flume-ng-sinks/flume-ng-hbase-sink/src/main/java/org/apache/flume/sink/hbase/HBaseSink.java e659ada flume-ng-sinks/flume-ng-hbase-sink/src/main/java/org/apache/flume/sink/hbase/HBaseSinkConfigurationConstants.java 5560624 Diff: https://reviews.apache.org/r/41529/diff/ Testing ------- Yes. Ran mvn clean install with no issues, no failures. File Attachments (updated) ---------------- patch 5 https://reviews.apache.org/media/uploaded/files/2015/12/28/1bc03ad3-6a7b-428b-993f-3a5558fc90fa__FLUME-2858-5.patch Thanks, Sam Glover
