tolitius / chazel

Hazelcast bells and whistles under the Clojure belt
Eclipse Public License 1.0
72 stars 8 forks source link

update hazelcast to 3.9 and add hz-list to retrieve a list from hazelcast #4

Closed hellonico closed 6 years ago

tolitius commented 6 years ago

thanks for PR. A couple of comments:

I did update chazel to 3.9.2 before, the reason I reverted back to 3.8.9 is because 3.9+ did not work (was very slow) in a couple of production clusters. Are there any particular features you are missing in 3.8.9?

hellonico commented 6 years ago

Exception in thread "hz._hzInstance_1_jet.cached.thread-4" java.lang.NoSuchMethodError: com.hazelcast.internal.partition.impl.InternalPartitionServiceImpl.isMigrationAllowed()Z at com.hazelcast.jet.impl.JobCoordinationService.shouldStartJobs(JobCoordinationService.java:551) at com.hazelcast.jet.impl.JobCoordinationService.scanJobs(JobCoordinationService.java:600) at com.hazelcast.spi.impl.executionservice.impl.DelegateAndSkipOnConcurrentExecutionDecorator$DelegateDecorator.run(DelegateAndSkipOnConcurrentExecutionDecorator.java:66) at com.hazelcast.util.executor.CachedExecutorServiceDelegate$Worker.run(CachedExecutorServiceDelegate.java:227)

... hence the update to something in the 3.9 series. I just realized today 3.9.3 was out , tried with that version and things worked smoothly when submitting jet jobs.

tolitius commented 6 years ago

sounds good, just fix the pom property and update hz to 3.9.3 I'll merge the pull request

hellonico commented 6 years ago

done !

tolitius commented 6 years ago

thank you