-----------------------------------------------------------
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

Reply via email to