Author Jim.Jewett
Recipients Jeremy Banks, Jim.Jewett, Mark.Bucciarelli, Ronny.Pfannschmidt, adamtj, asvetlov, aymeric.augustin, bulb, dholth, flox, ghaering, monsanto, pitrou, r.david.murray, scott.urban, torsten, tshepang, zzzeek
Date 2014-06-13.18:42:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1402684938.5.0.987801237961.issue10740@psf.upfronthosting.co.za>
In-reply-to
Content
(1)  The patch is just to docs, not code, so I'm not entirely sure that I understood it properly.

(2)  The new semantics are a huge mess to explain.  This might be because the old semantics were bad, but the result is the same.  I think it would be better to create a different API object.  (If I read correctly, you would leave function connect unchanged but also def dbapi_connect.)  Then keep this new one as simple as it should be -- which will probably mean leaving out some of the parameters that you deprecate.

(3)  The existing module documentation does claim to be DB-API 2.0 compliant; you should be explicit in the documentation (including docstring) about which behavior differences affect each of (database correctness/API standards conformance/ease of use).
History
Date User Action Args
2014-06-13 18:42:18Jim.Jewettsetrecipients: + Jim.Jewett, ghaering, pitrou, Jeremy Banks, r.david.murray, zzzeek, asvetlov, flox, adamtj, dholth, torsten, monsanto, scott.urban, aymeric.augustin, tshepang, Ronny.Pfannschmidt, Mark.Bucciarelli, bulb
2014-06-13 18:42:18Jim.Jewettsetmessageid: <1402684938.5.0.987801237961.issue10740@psf.upfronthosting.co.za>
2014-06-13 18:42:18Jim.Jewettlinkissue10740 messages
2014-06-13 18:42:18Jim.Jewettcreate