Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tune Cassandra options for faster startup #4309

Merged
merged 2 commits into from
Aug 30, 2021
Merged

Tune Cassandra options for faster startup #4309

merged 2 commits into from
Aug 30, 2021

Conversation

bsideup
Copy link
Member

@bsideup bsideup commented Jul 28, 2021

We never attempted at tuning Cassandra options to make it start faster. Apparently, there are a few low hanging fruits (focused on single-node scenario) that make it start significantly faster:

Before:

Container cassandra:3.11.2 started in PT32.628781S

After:

Container cassandra:3.11.2 started in PT10.720409S

Copy link
Member

@kiview kiview left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new heap defaults should also provide better OOTB support for some CI providers, correct?

@@ -58,8 +58,15 @@ public CassandraContainer(DockerImageName dockerImageName) {
dockerImageName.assertCompatibleWith(DEFAULT_IMAGE_NAME);

addExposedPort(CQL_PORT);
setStartupAttempts(3);
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was never necessary and only hid the problem

@bsideup
Copy link
Member Author

bsideup commented Jul 28, 2021

@kiview yep! (are you also thinking about BitBucket? ;))

Comment on lines +68 to +69
withEnv("HEAP_NEWSIZE", "128M");
withEnv("MAX_HEAP_SIZE", "1024M");
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by default, Cassandra will be allocating 10Gb of RAM or so - obviously not necessary for testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants