Hi Stefan,
On 25/12/2018 10:54, Stefan Seelmann wrote:
Hi all,
the replication tests were quite unstable: random failures and sometimes
they just hang forever.
While debugging I made some changes:
* In many tests we set the timeout for the LDAP API. Often it was set to
0 or -1 or Long.MAX_VALUE which means infinite wait time. In case of a
bug that means the test hangs forever. I removed all those, the default
timeout of 30 seconds applies now. Better fail than hang forever.
Yes, ity does not make any sense to wait forever. If teh connection is
down, it's up to the caller to deal with that.
* I also removed the infinite timeout in ReplicationConsumerImpl [1],
please let me know if there is a reason to keep that.
I'm pretty sure that was added for debugging purposes (it's hard to
debug when you just have 30 s to step through the code). Removing this
should have been done a long time ago...
The reason for the failures was a race condition in the LDAP API [2].
Good catch !
Many thanks Stefan !
