[ http://issues.apache.org/jira/brows...=3Dcomments#ac=
tion_12422645 ]=20
=20
Ersin Er commented on DIRSERVER-679:
------------------------------------

This patch, although a nice contribution, may need some reviewing. Here are=
a few points:

* The paths it updates are not relative to our repository but local file sy=
stem.
* There are some lines prefixed with - (minus) which means they are to be e=
xcluded, but they do not even exist in the current code base.
* Although it is targeted at 1.0-RC3, it has some Stored Procedure related =
stuff.

Well, I may not be %100 correct about my concerns but a review would be goo=
d.

Thanks Pierre!

> Tools improvement and add of Export Command and JUnit tests
> -----------------------------------------------------------
>
>                 Key: DIRSERVER-679
>                 URL: http://issues.apache.org/jira/browse/DIRSERVER-679
>             Project: Directory ApacheDS
>          Issue Type: Improvement
>          Components: tools
>    Affects Versions: 1.0-RC3
>            Reporter: Pierre-Arnaud Marcelot
>            Priority: Minor
>         Attachments: documentation.pdf, patch.txt
>
>
> Working on a new plugin supporting LDIF Import/Export for LDAP Studio ( h=
ttp://docs.safehaus.org/display/LDAPSTUDIO ), I wanted to use the Apache DS=
to import an LDIF file.
> But the Import Command can only be accessed via a Command Line and don't =
give enough feedback for a front-end GUI (particularly to show the progress=
of the operation).
> Emmanuel (L=C3=A9charny) also pointed out the fact that it may be interes=
ting for future developments to be able to access the Tools in a "kind of" =
Client/Server way. The tools running on a the server's station, and the com=
mand launched from another machine.
> So, he asked me to modify the tools to add this Client/Server feature and=
make them callable more easily from other application and give better feed=
back.
> I made a small documentation on this on Safehaus : http://docs.safehaus.o=
rg/display/LDAPSTUDIO/Apache+DS+Tools+rewrite . It is also attached to this=
post as a PDF, since LDAPStudio's Confluence isn't accessible to everyone.
> In the patch you'll find the tools with the new architecture (defined wit=
h Emmanuel).
> I also added a new command that export entries as LDIF file, and some JUn=
it tests for the Import, Export and GracefulShutdown command.
> Feel free to give me your feedback about this.

--=20
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: htt=
p://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

[ Post a follow-up to this message ]