Hi Dan !
You are damn right, this is a awfull piece of code. Hopefully, it's
never called anywhere. We have changed the code so that we are looking
DNs from their code (Long), not from their String value in the UpDN
table (not even sure that this table is still necessary).
Can you fill a JIRA so that we can remember to clean this code ?
Emmanuel
On 6/20/07, Barber, Dan (Contractor) <[EMAIL PROTECTED]> wrote:
All,
I'm working on going through BTreePartition, JdbmPartition, JdbmStore,
etc working on how we're going to make apacheds work with our existing
database. I'm making some progress in understanding. I THINK I came
across a bug in JdbmPartition, but wanted to ask the group, mostly to
verify that I'm not getting myself lost. The code snippet is
.
.
.
public final String getEntryUpdn( Long id ) throws NamingException
{
return store.getEntryUpdn( id );
}
public final String getEntryUpdn( String dn ) throws NamingException
{
return getEntryUpdn( dn );
}
public final int count() throws NamingException
{
return store.count();
}
.
.
.
I believe the code for getEntryUpdn should return store.getEntryUpdn( dn
) is that correct? I believe the current code is a stack overflow
waiting to happen.
Thanks,
Dan
--
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com
