5:55 p.m.
Every Firehose object uses __slots__ for memory optimization, but it
blocks tools like SQLAlchemy to add attributes dynamically; could we
remove this feature or render it optional?
--
Matthieu
7:25 p.m.
On Fri, 2013-07-05 at 17:55 +0200, Matthieu Caneill wrote:
Every Firehose object uses __slots__ for memory optimization, but it
blocks tools like SQLAlchemy to add attributes dynamically; could we
remove this feature or render it optional?
I ran into this myself when trying to write
https://github.com/davidmalcolm/firehose-orm
and commit 453f1944552bdc127b0778b34baa4f1d1c1e6b46 introduced the
Attribute classes to better capture this metadata.
I'm fine with the __slots__ being removed.
(the memory optimization is done for us on CPython 3.3+ and by PyPy.
There's a slight type-safety benefit to having them (since you can't
uses attrs other than those listed, which can help catch typos) - but
the benefit of having an ORM far outweighs these considerations)
Thanks
Dave
7:34 p.m.
On Fri, Jul 5, 2013 at 7:25 PM, David Malcolm <dmalcolm(a)redhat.com> wrote:
On Fri, 2013-07-05 at 17:55 +0200, Matthieu Caneill wrote:
> Every Firehose object uses __slots__ for memory optimization, but it
> blocks tools like SQLAlchemy to add attributes dynamically; could we
> remove this feature or render it optional?
I ran into this myself when trying to write
https://github.com/davidmalcolm/firehose-orm
and commit 453f1944552bdc127b0778b34baa4f1d1c1e6b46 introduced the
Attribute classes to better capture this metadata.
I'm fine with the __slots__ being removed.
(the memory optimization is done for us on CPython 3.3+ and by PyPy.
There's a slight type-safety benefit to having them (since you can't
uses attrs other than those listed, which can help catch typos) - but
the benefit of having an ORM far outweighs these considerations)
OK, cool!
Thank you :-)
--
Matthieu
11:01 a.m.
On Fri, Jul 5, 2013 at 7:34 PM, Matthieu Caneill
<matthieu.caneill42(a)gmail.com> wrote:
On Fri, Jul 5, 2013 at 7:25 PM, David Malcolm
<dmalcolm(a)redhat.com> wrote:
> On Fri, 2013-07-05 at 17:55 +0200, Matthieu Caneill wrote:
>> Every Firehose object uses __slots__ for memory optimization, but it
>> blocks tools like SQLAlchemy to add attributes dynamically; could we
>> remove this feature or render it optional?
>
> I ran into this myself when trying to write
> https://github.com/davidmalcolm/firehose-orm
> and commit 453f1944552bdc127b0778b34baa4f1d1c1e6b46 introduced the
> Attribute classes to better capture this metadata.
>
> I'm fine with the __slots__ being removed.
>
> (the memory optimization is done for us on CPython 3.3+ and by PyPy.
> There's a slight type-safety benefit to having them (since you can't
> uses attrs other than those listed, which can help catch typos) - but
> the benefit of having an ORM far outweighs these considerations)
OK, cool!
Thank you :-)
Pull request done here:
https://github.com/fedora-static-analysis/firehose/pull/18
Cheers.
Matthieu
4:25 p.m.
On Mon, 2013-07-08 at 11:01 +0200, Matthieu Caneill wrote:
On Fri, Jul 5, 2013 at 7:34 PM, Matthieu Caneill
<matthieu.caneill42(a)gmail.com> wrote:
> On Fri, Jul 5, 2013 at 7:25 PM, David Malcolm <dmalcolm(a)redhat.com> wrote:
>> On Fri, 2013-07-05 at 17:55 +0200, Matthieu Caneill wrote:
>>> Every Firehose object uses __slots__ for memory optimization, but it
>>> blocks tools like SQLAlchemy to add attributes dynamically; could we
>>> remove this feature or render it optional?
>>
>> I ran into this myself when trying to write
>> https://github.com/davidmalcolm/firehose-orm
>> and commit 453f1944552bdc127b0778b34baa4f1d1c1e6b46 introduced the
>> Attribute classes to better capture this metadata.
>>
>> I'm fine with the __slots__ being removed.
>>
>> (the memory optimization is done for us on CPython 3.3+ and by PyPy.
>> There's a slight type-safety benefit to having them (since you can't
>> uses attrs other than those listed, which can help catch typos) - but
>> the benefit of having an ORM far outweighs these considerations)
>
> OK, cool!
> Thank you :-)
Pull request done here:
https://github.com/fedora-static-analysis/firehose/pull/18
Pulled; thanks for doing this.
3966
days inactive
3969
days old
firehose-devel@lists.fedoraproject.org
4 comments
2 participants
participants (2)
-
David Malcolm -
Matthieu Caneill