Re: [Sqlalchemy-tickets] [sqlalchemy] #2720: new C extensions for attribute accessing

[sqlalchemy <mi...@zz...>]

#2720: new C extensions for attribute accessing
-----------------------------------+---------------------------------------
      Reporter:  zzzeek            |      Owner:  zzzeek
          Type:  enhancement       |     Status:  new
      Priority:  medium            |  Milestone:  blue sky
     Component:  cextensions       |   Severity:  very major - up to 2 days
    Resolution:                    |   Keywords:
Progress State:  not decided upon  |
-----------------------------------+---------------------------------------

Comment (by zzzeek):

 Unfortunately I can't give you a guarantee that I'll accept a patch which
 rewrites all the C extensions in Cython.  I'd need to see it, need to see
 in a bigger sense how the performance differences are (testing one or two
 functions in isolation is not that informative), need to understand how it
 works, how to maintain it, what kinds of build/installation issues are
 going to be introduced, where it might re-introduce mistakes that have
 been fixed in the existing C exts (relatively unlikely) or where it
 introduces all new mistakes (very likely).    I'm curious to see how
 they'd look but as I'm not the one playing with Cython right now I don't
 have too clear of a view how much code we'd introduce and how well it
 runs.

 Overall I'm not too enthused about a rewrite of the C exts at this point
 because we've got many years of stability with the current ones, they at
 least don't have any dependencies, and adding many more if any seems like
 a wasted effort - Pypy is a funded project going full steam ahead on
 making such C extensions unnecessary, and they are very interested most
 specifically in optimizing for the Django and SQLAlchemy projects.

--
Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2720#comment:29>
sqlalchemy <http://www.sqlalchemy.org/>
The Database Toolkit for Python