search

elodina / datastax-enterprise-mesos

DataStax Enterprise on Mesos
http://www.elodina.net
15 stars 4 forks source link

DSEM-14 - added option to pass cassandra process system props #65

Closed abiletskyi closed 8 years ago

dmitrypekar commented 8 years ago

Code is good. The only comment from me is following (sorry, for not pointing to this during initial discussion):

  1. I think, that the type of Node.cassandraJvmOptions should be changed to String, like in jvmOptions. In that case it would support more generic set of options, not only -D switches.
dmitrypekar commented 8 years ago

Merged. Thank you for the update.