[DBIx-Class-Devel] Re: [dbix-class] Add support for load_namespaces(lazy_load => 1, ...) (#32)

[Peter Rabbitson]

Well... allow me to explain I guess.

I am making this DBIC's problem because this is a simple "test my dependents" kind of problem. In this particular case it uncovered a bug in your implementation: since the user got this error: https://metacpan.org/source/ILMARI/Catalyst-Model-DBIC-Schema-0.61/lib/Catalyst/Model/DBIC/Schema.pm#L537 then it is obviously DBIC's fault since it is triggerd when your $schema->sources returns an empty list.

But these are details - the main point is that DBIC provides an API with certain guarantees. Lazy-loading is not allowed to break these guarantees unless absolutely justified. It would help greatly if you can do some of the triaging of problems that pop up for users of the alpha version. On the other hand your implied dismissal of a problem earlier in this thread as "it's out of DBIC's hands" was not at all constructive, which is why I had to reply in less uncertain terms ;)

Cheers

---
Reply to this email directly or view it on GitHub:
https://github.com/dbsrgits/dbix-class/pull/32#issuecomment-23847270
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.scsys.co.uk/pipermail/dbix-class-devel/attachments/20130904/043c9c07/attachment.htm