Hi Michael,

Nice work!

There are no major issues that I see with the patch. Can you please make sure that the internal tests as well as the tck tests run with and without security? And see if Michael can check in the code for you. If not, let me know and I'll do it.

In PersistenceManagerImpl can you remove the DEBUG flag?

I looked at the rest of the code and it looks good.

Craig

Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!

Attachment: smime.p7s
Description: S/MIME cryptographic signature



Reply via email to