search

bitsofinfo / hazelcast-docker-swarm-discovery-spi

Docker Swarm based discovery strategy SPI for Hazelcast enabled applications
Apache License 2.0
39 stars 33 forks source link

marked optional properties to prevent errors #5

Closed tunix closed 7 years ago

tunix commented 7 years ago

The plugin currently loads property values in 2 seperate ways:

  1. By providing system properties (aka java opts)
  2. By passing config properties

I'll send a PR for a unified solution for it but before that we need to make sure that optional properties are marked as optional. This is enforced by DefaultDiscoverService@187.