----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41529/ -----------------------------------------------------------
(Updated Dec. 28, 2015, 8:12 p.m.) Review request for Flume and Ted Malaska. 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. File Attachments ---------------- patch 5 https://reviews.apache.org/media/uploaded/files/2015/12/28/1bc03ad3-6a7b-428b-993f-3a5558fc90fa__FLUME-2858-5.patch patch 6 https://reviews.apache.org/media/uploaded/files/2015/12/28/fb74e4d9-f33c-49d0-be90-1781d6cc012c__FLUME-2858-6.patch Update 7 https://reviews.apache.org/media/uploaded/files/2015/12/28/fb6f5245-ec9b-4bcc-8552-0ff845aabd31__FLUME-2858-7.patch Thanks, Sam Glover
