Fixed zk storage matching

elodina / datastax-enterprise-mesos

DataStax Enterprise on Mesos

http://www.elodina.net

15 stars 4 forks source link

Fixed zk storage matching #91

Closed olegkovalenko closed 8 years ago

olegkovalenko commented 8 years ago

MOTIVATION: When zk storage specified with port Nodes#newStorage unable to match on case Array("zk", zk) because there are 3 elements in array after split (zk, master, 2181/dse-mesos).

PROPOSED CHANGE: Match only on "zk" prefix and concatenate rest of the array.

RESULT: ability to specify port for zk storage

dmitrypekar commented 8 years ago

Merged. Thank you.

olegkovalenko commented 8 years ago

Thanks!