Rlimit parameters are added to the Cassandra configuration.

[AnandJat]

As per documentation of Cassandra I keep default values as follows: <cassandra_user> - memlock unlimited (done set default -1) <cassandra_user> - nofile 1048576 (Not touched as its already implemented in master) <cassandra_user> - nproc 32768 ( done) <cassandra_user> - Stack (for stack its not mentioned in document, so I use default values on which service successfully UP and running) All these values are user configurable.

[mpereira]

Hi @AnandJat,

Thanks for the additional changes, they look good. I'm adding a do-not-merge to the resource limit PRs for now because we might have to do some more additional work, based on my findings from the last couple of days. I'll share more details tomorrow.

[kaiwalyajoshi]

@mpereira do we need to resurrect this? Also curious what the blockers were...

[rishabh96b]

Hi @kaiwalyajoshi , @AnandJat was saying that a planning was going on to migrate Cassandra from dcos-commons that is why it was left abandoned. I think at that time only elastic was migrated out as a standalone also.

[mpereira]

Hey @kaiwalyajoshi,

The blocker was the SDK bug I found when implementing this for Elastic. The SDK fix was committed only after my last comment in this PR, and released even later.

From what I see, @AnandJat's implementation should work now assuming that it uses an SDK version that contains the fix mentioned above. It will be best to make sure that all tests are passing before merging this, though.