----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41529/ -----------------------------------------------------------
(Updated Dec. 19, 2015, 1:31 p.m.) Review request for Flume and Ted Malaska. Changes ------- Added try / catch logic and resolved all compile, test issues when running "mvn clean install". 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 (updated) ----- 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. Thanks, Sam Glover
