Re: [ZODB-Dev] getExtensionMethods

2007-04-27 Thread Dieter Maurer
Jim Fulton wrote at 2007-4-26 15:44 -0400:
>For now, I'm leaving this feature in ZEO, but it is not part of any  
>official interfaces.  It isn't going to be part of the standard  
>storage API.  (I've removed it from the very annoying BaseStorage,  
>which really should be named SimpleStorage.) At some point, I'd still  
>like to hear about some specific use cases that motivate this feature.

When I remember right, then you will find a use case in
the mailing list archive.



-- 
Dieter
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] getExtensionMethods

2007-04-26 Thread Jim Fulton


For now, I'm leaving this feature in ZEO, but it is not part of any  
official interfaces.  It isn't going to be part of the standard  
storage API.  (I've removed it from the very annoying BaseStorage,  
which really should be named SimpleStorage.) At some point, I'd still  
like to hear about some specific use cases that motivate this feature.


Jim

On May 15, 2006, at 6:44 AM, Jim Fulton wrote:


Toby Dickenson wrote:

On Sunday 14 May 2006 21:52, Jim Fulton wrote:

Toby, this is almost certainly a question for you. :)

Can someone explain to me why getExtensionMethods is needed in the
storage API?  It's not used by anything in ZODB.

ZEO proxies the standard storage API.


Right.

The original requirement is for ZEO to support storages which  
expose extended functionality via an extended API.  
DirectoryStorage uses this to allow control of its snapshot mode.


I'm willing to be convinced that we need this, but without specific
arguments, I'm inclined to call YAGNI.


I'm incluned to remove it,
It would be mildly disappointing if there was no way of accessing  
an extended storage API over ZEO.


This is not exactly a ringing endorsement for something that you  
invented

and are the only one using.

Generality for it's own sake is not a convincing argument to me  
anymore.


> However I have no concerns about seeing
getExtensionMethods replaced with some alternative mechanism  
(possibly more complex for the consumer(s) of this extended API,  
if that makes it more sustainable within ZEO)

References:
First proposed here...
http://mail.zope.org/pipermail/zodb-dev/2002-May/002704.html


All this says is that you want this, but it doesn't say why.

Given that this has been around for a while, could you describe
how you've used it? Are you aware of anyone else who has?

Perhaps, it would be better to expand the storage API. I dunno.

I am concerned about a feature that no one but you uses, has  
effectively

no tests, and doesn't seem to be maintained.

Over the next few months I hope to put in a lot of time trying to  
clean up
ZODB.  Part of this effort will be cleaning out things that aren't  
needed

and giving what remains decent documentation and tests.

BTW, despite the number of excruciating ZEO tests, your's is not  
the only
untested feature.  On some level, I can sympathize with not writing  
tests
for ZEO features, as it's just so painful to do so.  I hope to fix  
this

over the next few months,

Jim

--
Jim Fulton   mailto:[EMAIL PROTECTED]   Python Powered!
CTO  (540) 361-1714http://www.python.org
Zope Corporation http://www.zope.com   http://www.zope.org



--
Jim Fulton  mailto:[EMAIL PROTECTED]Python 
Powered!
CTO (540) 361-1714  
http://www.python.org
Zope Corporationhttp://www.zope.com http://www.zope.org



___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] getExtensionMethods

2006-05-15 Thread Jim Fulton

Toby Dickenson wrote:

On Sunday 14 May 2006 21:52, Jim Fulton wrote:


Toby, this is almost certainly a question for you. :)

Can someone explain to me why getExtensionMethods is needed in the
storage API?  It's not used by anything in ZODB. 



ZEO proxies the standard storage API.


Right.

The original requirement is for ZEO to support storages which expose extended 
functionality via an extended API. DirectoryStorage uses this to allow 
control of its snapshot mode.


I'm willing to be convinced that we need this, but without specific
arguments, I'm inclined to call YAGNI.




I'm incluned to remove it,



It would be mildly disappointing if there was no way of accessing an extended 
storage API over ZEO.


This is not exactly a ringing endorsement for something that you invented
and are the only one using.

Generality for it's own sake is not a convincing argument to me anymore.

> However I have no concerns about seeing
getExtensionMethods replaced with some alternative mechanism (possibly more 
complex for the consumer(s) of this extended API, if that makes it more 
sustainable within ZEO)




References:
First proposed here...
http://mail.zope.org/pipermail/zodb-dev/2002-May/002704.html


All this says is that you want this, but it doesn't say why.

Given that this has been around for a while, could you describe
how you've used it? Are you aware of anyone else who has?

Perhaps, it would be better to expand the storage API. I dunno.

I am concerned about a feature that no one but you uses, has effectively
no tests, and doesn't seem to be maintained.

Over the next few months I hope to put in a lot of time trying to clean up
ZODB.  Part of this effort will be cleaning out things that aren't needed
and giving what remains decent documentation and tests.

BTW, despite the number of excruciating ZEO tests, your's is not the only
untested feature.  On some level, I can sympathize with not writing tests
for ZEO features, as it's just so painful to do so.  I hope to fix this
over the next few months,

Jim

--
Jim Fulton   mailto:[EMAIL PROTECTED]   Python Powered!
CTO  (540) 361-1714http://www.python.org
Zope Corporation http://www.zope.com   http://www.zope.org
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev


Re: [ZODB-Dev] getExtensionMethods

2006-05-15 Thread Toby Dickenson
On Sunday 14 May 2006 21:52, Jim Fulton wrote:
> 
> Toby, this is almost certainly a question for you. :)
> 
> Can someone explain to me why getExtensionMethods is needed in the
> storage API?  It's not used by anything in ZODB. 

ZEO proxies the standard storage API.

The original requirement is for ZEO to support storages which expose extended 
functionality via an extended API. DirectoryStorage uses this to allow 
control of its snapshot mode.

> I'm incluned to remove it,

It would be mildly disappointing if there was no way of accessing an extended 
storage API over ZEO. However I have no concerns about seeing 
getExtensionMethods replaced with some alternative mechanism (possibly more 
complex for the consumer(s) of this extended API, if that makes it more 
sustainable within ZEO)



References:
First proposed here...
http://mail.zope.org/pipermail/zodb-dev/2002-May/002704.html
and an implementation discussed in the thread starting here...
http://mail.zope.org/pipermail/zodb-dev/2002-September/003209.html



-- 
Toby Dickenson
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev


[ZODB-Dev] getExtensionMethods

2006-05-14 Thread Jim Fulton


Toby, this is almost certainly a question for you. :)

Can someone explain to me why getExtensionMethods is needed in the
storage API?  It's not used by anything in ZODB.  I'm incluned to
remove it, but I imagine that it's there for a reason.  It is
plumbed through ZEO with no tests.

Jim

--
Jim Fulton   mailto:[EMAIL PROTECTED]   Python Powered!
CTO  (540) 361-1714http://www.python.org
Zope Corporation http://www.zope.com   http://www.zope.org
___
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev