[Zope3-dev] Re: CVS: Zope3/src/zope/app/services/tests - test_connectionservice.py:1.13 test_connectionregistration.py:NONE

[Jim Fulton]

Stephan Richter wrote:
> Update of /cvs-repository/Zope3/src/zope/app/services/tests
> In directory cvs.zope.org:/tmp/cvs-serv15967/services/tests
> 
> Modified Files:
> 	test_connectionservice.py 
> Removed Files:
> 	test_connectionregistration.py 
> Log Message:
> Heads up, this is the beginning of Servicegeddon. We noticed that named 
> components are a pain in the butt and that we can simpluify our code a lot
> by making these named components local utilities. 
> 
> So I started with the SQL Connection service and database adapters and 
> simplified the code greatly. Database Adaptors are now just simple 
> utilities.
> 
> I also took the oppurtunity and updated all database adapters. 
> 
> This change is not backward compatible and you will have to toss your ZODB.
> Although all tests pass, I expect there to be some hickups, so feel free to 
> fix or report these. 
> 
> Next I will fix up the front screen a bit and try to factor some other 
> pieces. I also still miss some functionality in the Utilities, which I plan
> to add.

Whoa,

You can't just tell people that they need to toss their databases.

You need to provide a conversion script.

Please don't check in backward incompatible changes without providing a
conversion script.

You need to stop and work with folks on this list to find out if
it'a acceptable to toss the database. If it's not, you need to back the change
out until you can provide a conversion script.

Jim



-- 
Jim Fulton           mailto:jim at zope.com       Python Powered!
CTO                  (703) 361-1714            http://www.python.org
Zope Corporation     http://www.zope.com       http://www.zope.org