Re: [Sqlalchemy-tickets] [sqlalchemy] #2161: Py3k support for the C extension
Brought to you by:
zzzeek
|
From: sqlalchemy <mi...@zz...> - 2013-07-04 17:41:59
|
#2161: Py3k support for the C extension
-------------------------------+-------------------------------
Reporter: ged | Owner: pjenvey
Type: enhancement | Status: new
Priority: medium | Milestone: 0.9.xx
Component: cextensions | Severity: major - 1-3 hours
Resolution: | Keywords:
Progress State: needs review |
-------------------------------+-------------------------------
Comment (by zzzeek):
Reimplementing in Cython is "fine" as a long term goal, however to get our
existing C extensions to work in py3k should be a far smaller task. Not
just in terms of coding. More importantly, in terms of tests and proven
stability; the C extensions we have right now are very widely used and
tested in thousands if not tens of thousands of installations. If we
rewrite C exts from scratch using a new system, we throw that maturity out
the window and start all over again.
Adding C extensions for small functions here and there isn't really going
to get us much in terms of performance, and I'm more or less -1 on the
#2720 improvement as far as attribute access, the ratio of code
verbosity/complexity to performance gain is too high in that case. In
order to break out of what are very diminishing returns right now, we'd
really need to reconstruct the entire core and ORM using C throughout.
This would definitely be more of a Cython project as it would be
unworkable otherwise, but that's a large and distant undertaking. Right
now, Pypy continues to be developed with funding, suggesting the irony
that the longer we do nothing, the more progress is actually made towards
us getting all the speed improvements via pypy for free.
As far as this issue, while it's a small task, it's been sitting here for
literally two years, I'll likely be digging in to learn the API
differences and try to get this in for 0.9.
--
Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2161#comment:21>
sqlalchemy <http://www.sqlalchemy.org/>
The Database Toolkit for Python
|