-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Description
Request motivated by the fact that we simply don't have need for it.
There are a few simple and one complex change involved:
- Make
HazelCastConfigrequire a profile for activation. Very straightforward, just add@Profile("hazelcast")or somesuch. - Instruct users to add
spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.hazelcast.HazelcastAutoConfigurationinto application configuration if they wish to disable the feature. - Instead of autowiring an instance of
HazelcastInstancein application components abstract the needed functionality into an interface and convert the Hazelcast integration into a helper module structure, possibly underrutebanken-helpersfollowing the modularization logic there.
It's worth noting that there already is rutebanken-helper/hazelcast4-helper and the code in tiamat seems to be a direct match with that one. Looking at the version history things have diverged, and the helper in fact has some relevant, useful changes in it already.
Reactions are currently unavailable