These are chat archives for atomix/atomix
We are no longer monitoring this channel, please join Slack! https://join.slack.com/t/atomixio/shared_invite/enQtNDgzNjA5MjMyMDUxLTVmMThjZDcxZDE3ZmU4ZGYwZTc2MGJiYjVjMjFkOWMyNmVjYTc5YjExYTZiOWFjODlkYmE2MjNjYzZhNjU2MjY
LocalTransport(which the test stuff uses) it's not really well suited for testing since it doesn't exactly mimic the behavior of
LocalTransportdoesn't do timeouts. If an exception occurs - which sometimes happens during shutdown because
RejectedExecutionExceptionis thrown by executors - then often a request just never finishes and it hangs.
LocalTransportis really just good for what it's used for in Atomix. What's needed is a legitimate
TestTransportor tests should just use the
TestTransport. The problem with
LocalTransportis that it's legitimately used in
AtomixReplicato send messages directly to the local server without using the loopback interface. So, because of that it doesn't do serialization or timeouts since serialization is expensive and timeouts are unnecessary in the replica. We need a
TestTransportthat more closely mimics
NettyTransportif we're not using networks in some tests.