Hi Carl, On Wednesday 22 October 2014 19:35:49 Carl Meyer wrote: > On 10/21/2014 04:04 PM, Shai Berger wrote: > > I'd argue that in the common case, the user shouldn't care if the > > function they are calling is implemented in Python or Procedural SQL > > (assuming it is going to interact with the database either way), and so > > it is good API design to abstact this detail away. > > Here is where we differ. I think calling a database stored procedure is > fundamentally a different thing from calling a Python function (even one > that accesses the database), and the difference should be obvious in the > calling code. I don't think it is good API design to abstract away > differences that the caller should be aware of. (For one example of a > difference, database stored procedures "return a modified copy of the > input arguments", which would be extremely unusual behavior for a normal > Python function.) > That, of course, doesn't have to be the case for the wrapper API.
> I think [...] at > the level of Django core the benefits of a generic API that attempts to > mask the difference between Python functions and database stored > procedures are not enough to justify the magic required. > > Perhaps I'm wrong - but I'd definitely want to see the utility of such a > generic "magic" layer proved as an external library before it gets added > to core. > I agree that this would be the prudent step to take. On second thought, I expect the benefit to prove quite small, just because people don't use all that many stored procedures in Django apps. Shai. -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/201410230256.13570.shai%40platonix.com. For more options, visit https://groups.google.com/d/optout.
