Merge lp://staging/~stefanor/ibid/sqlalchemy-0.6-trunk into lp://staging/ibid

Proposed by Stefano Rivera
Status: Merged
Approved by: Stefano Rivera
Approved revision: 1038
Merged at revision: 1052
Proposed branch: lp://staging/~stefanor/ibid/sqlalchemy-0.6-trunk
Merge into: lp://staging/ibid
Diff against target: 847 lines (+187/-158)
20 files modified
docs/install.rst (+1/-1)
ibid/core.py (+42/-34)
ibid/db/__init__.py (+2/-5)
ibid/db/types.py (+62/-50)
ibid/db/versioned_schema.py (+20/-24)
ibid/plugins/codecontest.py (+1/-1)
ibid/plugins/factoid.py (+6/-6)
ibid/plugins/feeds.py (+1/-1)
ibid/plugins/fun.py (+3/-3)
ibid/plugins/identity.py (+13/-13)
ibid/plugins/karma.py (+12/-5)
ibid/plugins/memo.py (+3/-3)
ibid/plugins/seen.py (+1/-1)
ibid/plugins/urlgrab.py (+1/-1)
ibid/test/__init__.py (+10/-1)
scripts/ibid-factpack (+2/-2)
scripts/ibid-knab-import (+3/-3)
scripts/ibid-plugin (+1/-1)
scripts/ibid-setup (+2/-2)
setup.py (+1/-1)
To merge this branch: bzr merge lp://staging/~stefanor/ibid/sqlalchemy-0.6-trunk
Reviewer Review Type Date Requested Status
Keegan Carruthers-Smith Approve
Stefano Rivera Approve
Jonathan Hitchcock Approve
Max Rabkin Approve
Review via email: mp+66033@code.staging.launchpad.net

This proposal supersedes a proposal from 2010-06-16.

Commit message

The massive SQLALchemy 0.6 + 0.7 patch. Only 18 months late :)
Main points:
* Use SQLALchemy 0.6 APIs, dropping 0.5 support.
* Create fullblown SQLAlchemy types for IbidUnicode(Text) rather than use TypeDecorator.

Description of the change

We should be talking about merging this

To post a comment you must log in.
Revision history for this message
Max Rabkin (max-rabkin) wrote : Posted in a previous version of this proposal

bucket now uses the database

review: Needs Fixing
Revision history for this message
Keegan Carruthers-Smith (keegan-csmith) wrote : Posted in a previous version of this proposal

Don't know much about sqlalchemy, so gonna abstain.

review: Abstain
Revision history for this message
Stefano Rivera (stefanor) wrote :

I see from uncommitted diffs on my laptop, that I started trying to get to the bottom of the test failures with SQLAlchemy > 0.5, but I think that's non-trivial.

How about we land this now, and fix it up in trunk?

Revision history for this message
Keegan Carruthers-Smith (keegan-csmith) wrote :

Approving this because ibid has been running this branch for a while, and there have not been any problems. I did do a quick look over the diff and couldn't see any problems.

review: Approve
Revision history for this message
Keegan Carruthers-Smith (keegan-csmith) wrote :

by ibid, i mean tibid.

Revision history for this message
Ibid Branch Auto-Lander (ibid-tarmac) wrote :

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0. Got: 2 Pending.

Revision history for this message
Max Rabkin (max-rabkin) wrote :

per Keegan

review: Approve
Revision history for this message
Ibid Branch Auto-Lander (ibid-tarmac) wrote :
Download full text (3.4 KiB)

The attempt to merge lp:~stefanor/ibid/sqlalchemy-0.6-trunk into lp:ibid failed. Below is the output from the failed tests.

                                                                        [ERROR]

===============================================================================
[ERROR]: ibid

Traceback (most recent call last):
  File "/usr/lib/python2.6/dist-packages/twisted/trial/runner.py", line 651, in loadByNames
    things.append(self.findByName(name))
  File "/usr/lib/python2.6/dist-packages/twisted/trial/runner.py", line 460, in findByName
    return filenameToModule(name)
  File "/usr/lib/python2.6/dist-packages/twisted/trial/runner.py", line 95, in filenameToModule
    ret = reflect.namedAny(reflect.filenameToModuleName(fn))
  File "/usr/lib/python2.6/dist-packages/twisted/python/reflect.py", line 464, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/stefanor/ibid/tarmac-trunk/ibid/__init__.py", line 16, in <module>
    import ibid.core
  File "/home/stefanor/ibid/tarmac-trunk/ibid/core.py", line 21, in <module>
    from ibid.db import SchemaVersionException, schema_version_check
  File "/home/stefanor/ibid/tarmac-trunk/ibid/db/__init__.py", line 5, in <module>
    from ibid.db.types import Integer, DateTime, Boolean, \
  File "/home/stefanor/ibid/tarmac-trunk/ibid/db/types.py", line 75, in <module>
    monkey_patch()
  File "/home/stefanor/ibid/tarmac-trunk/ibid/db/types.py", line 9, in monkey_patch
    import sqlalchemy.dialects.postgresql
exceptions.ImportError: No module named dialects.postgresql
-------------------------------------------------------------------------------

FAILED (errors=1)

Traceback (most recent call last):
  File "/usr/lib/python2.6/dist-packages/twisted/python/usage.py", line 241, in parseOptions
    self.postOptions()
  File "/usr/lib/python2.6/dist-packages/twisted/scripts/trial.py", line 295, in postOptions
    self['reporter'] = self._loadReporterByName(self['reporter'])
  File "/usr/lib/python2.6/dist-packages/twisted/scripts/trial.py", line 281, in _loadReporterByName
    for p in plugin.getPlugins(itrial.IReporter):
  File "/usr/lib/python2.6/dist-packages/twisted/plugin.py", line 200, in getPlugins
    allDropins = getCache(package)
--- <exception caught here> ---
  File "/usr/lib/python2.6/dist-packages/twisted/plugin.py", line 165, in getCache
    provider = pluginModule.load()
  File "/usr/lib/python2.6/dist-packages/twisted/python/modules.py", line 381, in load
    return self.pathEntry.pythonPath.moduleLoader(self.name)
  File "/usr/lib/python2.6/dist-packages/twisted/python/reflect.py", line 464, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/stefanor/ibid/tarmac-trunk/twisted/plugins/ibid_plugin.py", line 10, in <module>
    import ibid
  File "/home/stefanor/ibid/tarmac-trunk/ibid/__init__.py", line 16, in <module>
    import ibid.core
  File "/home/stefanor/ibid/tarmac-trunk/ibid/core.py", line 21, in <module>
    from ibid.db import SchemaVersionException, schema_version_check
  File "/home/stefanor/ibid/tarmac-trunk/ibid/db/__init__.py", line 5, in <module>
    from ibid.db.types import Integer, DateTime, Boolean, \
 ...

Read more...

Revision history for this message
Jonathan Hitchcock (vhata) wrote :

Surewhynot.

review: Approve
Revision history for this message
Ibid Branch Auto-Lander (ibid-tarmac) wrote :

Attempt to merge into lp:ibid failed due to conflicts:

text conflict in ibid/test/__init__.py

Revision history for this message
Ibid Branch Auto-Lander (ibid-tarmac) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Stefano Rivera (stefanor) wrote :

Here's a biscuit, tarmac

review: Approve
Revision history for this message
Keegan Carruthers-Smith (keegan-csmith) wrote :

blind approvals ftw

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: