I'm a long term user of this module, even before it's existence on CPAN. This module works good for me.
Why only 3 stars?
Generally it's good that this module is now on CPAN after it was hidden from most of the world on a japanese site. Unfortunately there are some issues with it:
- The file should use a lib path that matches its namespace, i.e., lib/DBIx/Class/Schema/Loader/DBI/Oracle.pm.
- It should be merged into the DBIx-Class-Schema-Loader package. In fact it already exists there but is hidden from pause indexer. (Strangely this only works because of it's original wrong lib path, see first point above.)
- It has documentation issues, eg. it uses "Postgres" in its description, probably because it was forked from that package. It's not that critical but such things make the user unsure. The hidden instance in DBIx-Class-Schema-Loader already fixed that.
My suggestion: Merge and fix the documentation (eg. the DBD::Oracle speed hint) into the DBIx-Class-Schema-Loader instance, no longer hide that other instance from indexer and take this instance here away again from CPAN.
I know, that it's a pity to give up on modules for the sake of authors reputation, but having it central in DBIx-Class-Schema-Loader is more worthy for the users. Maybe just incrementing version number in the DBIx-Class-Schema-Loader instance solves the reputation problem.
4 out of 4 found this review helpful. Was this review helpful to you? Yes No