Chronicle-Network

Chronicle-Network copied to clipboard

Deprecate SEU once IOTools is released https://github.com/OpenHFT/Chronicle-Core/issues/436 @nicktindall fyi

peter-lawrey

It looks like the setting to reject the handler is being ignored.

peter-lawrey

``` java.lang.AssertionError: Closeables still open at net.openhft.chronicle.core.io.AbstractCloseable.assertCloseablesClosed(AbstractCloseable.java:160) at net.openhft.chronicle.core.io.AbstractReferenceCounted.assertReferencesReleased(AbstractReferenceCounted.java:61) at net.openhft.chronicle.network.NetworkTestCommon.assertReferencesReleased(NetworkTestCommon.java:27) at net.openhft.chronicle.network.NetworkTestCommon.afterChecks(NetworkTestCommon.java:74) at jdk.internal.reflect.GeneratedMethodAccessor5.invoke(Unknown Source) Suppressed: java.lang.IllegalStateException: Not closed IdempotentLoopStartedEventHandler@1 closed=false at net.openhft.chronicle.core.io.AbstractCloseable.assertCloseablesClosed(AbstractCloseable.java:184) ... 31 more Caused by: net.openhft.chronicle.core.StackTrace:...

alamar

jkolobok

(when using the UberHandler at least) It's called here https://github.com/OpenHFT/Chronicle-Network/blob/5d53d322e4b1713c6e8ce24326038f4e382a0b53/src/main/java/net/openhft/chronicle/network/cluster/handlers/UberHandler.java#L157 and here https://github.com/OpenHFT/Chronicle-Network/blob/5d53d322e4b1713c6e8ce24326038f4e382a0b53/src/main/java/net/openhft/chronicle/network/cluster/handlers/HeartbeatHandler.java#L227-L228

nicktindall

Replace the java socket classes with delegating interfaces so that the native java can be replaced with optimal C++ networking code

1

RobAustin

AbstractStatelessClient::toParameters is using double dynamic lambda

1

This creates unnecessary lambda instantiations for calls with two or more parameters.

minborg

alamar