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

Allow to pass instances of ClientConfig and Config to Hazelcast data store factory #8600

Merged
merged 3 commits into from
Sep 21, 2022

Conversation

Vlatombe
Copy link
Contributor

This increases flexibility when configuring it programmatically.

Patterns such as

HazelcastSessionDataStoreFactory f = new HazelcastSessionDataStoreFactory();
f.setOnlyClient(true);
f.setClientConfig(ClientConfig.load());

or

HazelcastSessionDataStoreFactory f = new HazelcastSessionDataStoreFactory();
f.setServerConfig(Config.load());

can then be used to configure Hazelcast according to standard Hazelcast locations if needed.

…store factory

This increases flexibility when configuring it programmatically.

Patterns such as

HazelcastSessionDataStoreFactory f = new HazelcastSessionDataStoreFactory();
f.setOnlyClient(true);
f.setClientConfig(ClientConfig.load());

or

HazelcastSessionDataStoreFactory f = new HazelcastSessionDataStoreFactory();
f.setServerConfig(Config.load());

can then be used to configure Hazelcast according to standard Hazelcast
locations if needed.
@olamy
Copy link
Member

olamy commented Sep 20, 2022

@Vlatombe thanks for the contribution.
Can you please first sign the Eclipse ECA?
See details here https://www.eclipse.org/legal/ECA.php (remember to use same email as the one used with your commit)
thanks

@Vlatombe
Copy link
Contributor Author

I signed it with the email address matching the commit, but my Eclipse.org account was associated with another email address (which I changed). However the check still doesn't detect it.

@olamy
Copy link
Member

olamy commented Sep 21, 2022

I signed it with the email address matching the commit, but my Eclipse.org account was associated with another email address (which I changed). However the check still doesn't detect it.

no worries. there might be some caching in the Eclipse infra for validation.
All good now.
Thanks!

@olamy olamy merged commit 090104d into jetty:jetty-10.0.x Sep 21, 2022
@Vlatombe Vlatombe deleted the hazelcast-config branch September 26, 2022 07:21
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.

3 participants