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

CLOUD-4211 Cloud clustering profiles should configure a distributed cache instead of a replicated one #404

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

rhusar
Copy link
Member

@rhusar rhusar commented May 21, 2024

Currently, our cloud tooling configures clustering in the cloud using a replicated cache mode which is suboptimal as it does not scale. Such cache mode does not scale and is not suited for applications in the cloud that are supposed to be scalable. In this mode Infinispan replicates all entries in the cache to all nodes in the cluster. It doesn't scale, as adding new nodes does not enlarge the cluster capacity. Moreover, we should relax the isolation level to READ_COMMITTED (from REPEATABLE_READ) and transaction mode to NONE (from BATCH).

Resolve
https://issues.redhat.com/browse/CLOUD-4211

@rhusar rhusar changed the title CLOUD-4211 Cloud clustering profiles should configure a distributed c… CLOUD-4211 Cloud clustering profiles should configure a distributed cache instead of a replicated one May 21, 2024
@jfdenise
Copy link
Contributor

@rhusar rhusar force-pushed the CLOUD-4211 branch 3 times, most recently from 52ee61e to 5351a42 Compare June 10, 2024 09:34
…ache instead of a replicated one - introduce 'sessions' cache to contain HTTP session data.
…ache instead of a replicated one - introduce 'beans' cache to contain SFSB session data.
@rhusar rhusar marked this pull request as ready for review June 11, 2024 14:53
@rhusar rhusar requested a review from jfdenise as a code owner June 11, 2024 14:53
@jfdenise jfdenise merged commit 72f8fab into wildfly:main Jun 26, 2024
3 checks passed
@jfdenise
Copy link
Contributor

@rhusar , merged. Thank-you. The wildfy-cloud-galleon-pack 7.0.1.Final will to contain this fix.

@jfdenise
Copy link
Contributor

@rhusar rhusar deleted the CLOUD-4211 branch June 27, 2024 20:38
@rhusar
Copy link
Member Author

rhusar commented Jun 27, 2024

Excellent, thanks! Will test.

@rhusar
Copy link
Member Author

rhusar commented Jun 27, 2024

@rhusar , do we really need to add the web dist cache? That is the default in the ha wildfly configuration: https://github.com/wildfly/wildfly/blob/main/ee-feature-pack/galleon-shared/src/main/resources/layers/standalone/web-clustering/layer-spec.xml#L20 and https://github.com/wildfly/wildfly/blob/main/ee-feature-pack/galleon-shared/src/main/resources/feature_groups/infinispan-dist-web.xml#L10 Could we just remove all this adjustment?

I forgot to reply to this one – but long story short, we wanted to change the configuration and finally rename the caches to something more reasonable as described on the Jira in more detail.

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

Successfully merging this pull request may close these issues.

3 participants