Issue #539 has been updated by David Coutadeur.

quickstart hsqldb tested successfully with my patch...
Maybe more test should be done with different kind of async ldap sources / 
database dest...
----------------------------------------
Bug #539: bug when ldap async source service + database dest service (lsc 2.0, 
trunk)
http://tools.lsc-project.org/issues/539

Author: David Coutadeur
Status: New
Priority: Normal
Assigned to: 
Category: Core
Target version: 2.0.2
Problem in version: 2.0


Hi,
Here is the bug when launch an asynchronous ldap source to a database 
destination service task :
(the problem does not appear for a synchronous task)

oct. 24 20:26:01 - ERROR - Error while synchronizing ID
uidInterne=myuser,ou=People,dc=organization,dc=org:
java.lang.UnsupportedOperationException: This method should never be
called - this is a software BUG ! 

Interesting debugging stuff here :
- SimpleJdbcDstService, line 117 and after
- AbstractJdbcService, line 193 and after
- AbstractSynchronize, line 707 and after

AbstractSynchronize launching this :
dstBean = task.getDestinationService().getBean(id.getKey(), id.getValue(), 
true);
for a synchronous task
and this :
dstBean = task.getDestinationService().getBean(entry.getMainIdentifier(), 
entryDatasets, false);
for an asynchronous task

then the getBean is launching :
List<?> records = sqlMapper.queryForList((fromSource ? 
getRequestNameForObject() : getRequestNameForClean()), 
getAttributesMap(attributes));

so it launches getRequestNameForObject for a synchronous task and 
getRequestNameForClean for an asynchronous task.
The getRequestNameForClean should never be called, and send an error.

I made a patch to solve this, which always call the getBean with the true 
parameter.

However, I don't know all the implication of this. Especially database source 
-> ldap dest, async + sync. If somebody could look at this and :
1/ test in an exhaustive way
or,
2/ look at the code with a global and exhaustive approache to confirm I am not 
doing wrong...

David


-- 
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