[tahoe-lafs-trac-stream] [Tahoe-LAFS] #2373: adopt pyrsistent

[Tahoe-LAFS]

#2373: adopt pyrsistent
-----------------------------+-----------------------
     Reporter:  zooko        |      Owner:
         Type:  enhancement  |     Status:  new
     Priority:  normal       |  Milestone:  undecided
    Component:  unknown      |    Version:  1.10.0
   Resolution:               |   Keywords:
Launchpad Bug:               |
-----------------------------+-----------------------

Comment (by exarkun):

 pyrsistent is quite slow without the C extensions.  Even with them,
 libraries like Eliot have observed that a lot of time ends up spent in
 pyrsistent implementation (I have an idea that for Eliot, native CPython
 types were found to be faster but I can't produce evidence so don't assign
 this much weight).

 The big downsides discussed above have been neutralized at this point as
 Tahoe-LAFS now already has pyrsistent as a transient dependency.  I
 haven't noticed any particular problems arising from this.  I think that
 binary wheels are now supported well-enough by the relevant parts of the
 install toolchain that simple extension modules like this are not really a
 big deal any more (on the other hand, it could mean that few newer users
 are adopting Tahoe-LAFS these days and everyone else has long since
 figured out how to keep it working).

 Anyway, I think this should probably be done.  That said, someone has to
 actually rewrite some part of Tahoe-LAFS to make use of pyrsistent instead
 of some other data structures...

--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2373#comment:2>
Tahoe-LAFS <https://Tahoe-LAFS.org>
secure decentralized storage