Issue #464 has been updated by Clément OUDOT.

Category set to Core
Assigned to set to Raphaël Ouazana
Target version set to 2.1


----------------------------------------
Bug #464: IAsynchronousService is not so (a)synchronous
http://tools.lsc-project.org/issues/464

Author: Raphaël Ouazana
Status: New
Priority: Normal
Assigned to: Raphaël Ouazana
Category: Core
Target version: 2.1
Problem in version: 


First this service should be named ISynchronousService. But this would be a 
very big and long refactoring, so let's admit that ASynchronous means 
synchronous.

The real problem is that unlike specified in the Javadoc, the getNextId() 
function is not blocking at all.
Eg. in the SyncReplSourceService:
<pre>searchResponse = sf.get(1, TimeUnit.NANOSECONDS);</pre>
So the call is only blocking for 1 ns, which is not really blocking (and it 
seems there is a bug in the Apache API: the TimeoutException is never thrown).

The real blocking is done in AbstractSynchronize.run():
<pre>Thread.sleep(aSrcService.getInterval() * 1000);</pre>

So the thread is not blocking until an event occurs, but systematically (by 
default) 5 seconds.

I think a timeout parameter should be handled directly by the 
SyncReplSourceService to make it really blocking.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://tools.lsc-project.org/my/account
_______________________________________________________________
Ldap Synchronization Connector (LSC) - http://lsc-project.org

lsc-dev mailing list
[email protected]
http://lists.lsc-project.org/listinfo/lsc-dev

Reply via email to