Roundup Tracker - Issues

Message4852

Author rouilj
Recipients ThomasAH, ber, ezio.melotti, rouilj, schlatterbeck
Date 2013-04-23.20:31:57
Message-id <201304232031.r3NKVtxl006013@mx1.cs.umb.edu>
In-reply-to Your message of "Tue, 23 Apr 2013 19:26:52 -0000." <1366745212.66.0.999759047942.issue2550731@psf.upfronthosting.co.za> <1366745212.66.0.999759047942.issue2550731@psf.upfronthosting.co.za>
In message
<1366745212.66.0.999759047942.issue2550731@psf.upfronthosting.co.za>
<1366745212.66.0.999759047942.issue2550731@psf.upfronthosting.co.za>,
Bernhard Reiter writes:

>Bernhard Reiter added the comment:
>
>John, adding a call to " self.db.post_init()" after adding the
>properties to the objects make the tests run fine for me. See
>the example in db_test_base.py.

I'll try it tonight when I get home. I know I added db.commit() and I
am pretty sure I used db.post_init() (following the code in
db_test_base.py:setupSchema) as well.

Maybe I just didn't have the schema changes and the post_init call
arranged properly to work.

For those following along at home from hyperdb.py:

     def post_init(self):
        """Called once the schema initialisation has finished.
           If 'refresh' is true, we want to rebuild the backend
           structures.
        """
        raise NotImplementedError

The version in anydbm (roundup/backends/back_anydbm.py using the db
file structure) only reindexes data. It doesn't have any code that
handles schema changes. Since I had no data to reindex at that point,
this test passed.

However post_init from roundup/backends/rdbms_common.py which is used
by postgres and sqlite does update tables and things based on schema
changes hence the reason the sqlite and postgres tests failed.

Thanks for looking at this Bernhard.

Hopefully there will be a commit coming soon.
History
Date User Action Args
2013-04-23 20:31:57rouiljsetrecipients: + rouilj, schlatterbeck, ber, ThomasAH, ezio.melotti
2013-04-23 20:31:57rouiljlinkissue2550731 messages
2013-04-23 20:31:57rouiljcreate