-
Notifications
You must be signed in to change notification settings - Fork 33
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
Constructor overload that includes swarmMgrUri #22
Comments
Actually, it might make more sense to set these specific settings through environment variables... Considering they aren't really application settings, but rather settings related to the Swarm cluster. I'll think about this for a while, then maybe close the issue. |
having it set via constructor option is not bad either, as long as still works as-is, please submit a PR and will put it in, won't hurt |
Yeah, if I need the constructor, I'll gladly submit a PR! However, as I was thinking about setting them all through environment variables, I noticed that only the swarm manager URI can be set through environment variables. So that plans goes out the window as well. :( What would you think about allowing setting them all through environment variables? For example, first checking the system properties. If they aren't set, check environment variables. If they aren't set, it will obviously fail. EDIT: Actually, constructor is good. I can handle the environment variables myself! |
Sure, please submit a PR to make all the configurable probs editable via env vars. Thanks! |
I find it strange that all properties can be set through a constructor of
SwarmMemberAddressProvider
exceptswarmMgrUri
(andskipVerifySsl
, but that is of less importance). It can only be set through an environment variable. I would like to keep my settings in one place, and I'm currently creating my Hazelcast config programmatically, like this:... and there's no way of setting the swarm manager URI. :(
The text was updated successfully, but these errors were encountered: