Merge lp://staging/~gerdusvanzyl/storm/firebird into lp://staging/storm

Proposed by Gerdus van Zyl
Status: Needs review
Proposed branch: lp://staging/~gerdusvanzyl/storm/firebird
Merge into: lp://staging/storm
Diff against target: 587 lines (+427/-27)
4 files modified
storm/databases/firebird.py (+229/-0)
tests/databases/base.py (+25/-25)
tests/databases/firebird.py (+171/-0)
tests/databases/proxy.py (+2/-2)
To merge this branch: bzr merge lp://staging/~gerdusvanzyl/storm/firebird
Reviewer Review Type Date Requested Status
Storm Developers Pending
Storm Developers Pending
Review via email: mp+28559@code.staging.launchpad.net

Description of the change

Basic Firebird database support.

Passes all tests except "Connection.rollback() swallows disconnect errors" which I have so far been unable to fix.

example connection string env variable for tests: STORM_FIREBIRD_URI=firebird://sysdba:masterke@localhost:3050/stormtest.fdb

To post a comment you must log in.
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

Hi Gerdus,

Thanks for pushing this forward into a request!

Sorry if this has been asked already, but if not, can you please sign the contributor agreement at:

    http://www.canonical.com/contributors

Revision history for this message
Gerdus van Zyl (gerdusvanzyl) wrote :

No problem, I have just sent the email as per the instructions.

On Tue, Aug 10, 2010 at 7:23 PM, Gustavo Niemeyer <email address hidden> wrote:
> Hi Gerdus,
>
> Thanks for pushing this forward into a request!
>
> Sorry if this has been asked already, but if not, can you please sign the contributor agreement at:
>
>    http://www.canonical.com/contributors
> --
> https://code.launchpad.net/~gerdusvanzyl/storm/firebird/+merge/28559
> You are the owner of lp:~gerdusvanzyl/storm/firebird.
>

--
Gerdus van Zyl

Unmerged revisions

368. By Gerdus van Zyl

merge from trunk

367. By Gerdus van Zyl

cleanup unused comments

366. By Gerdus van Zyl

fixed proxy to work on windows
replaced os.read(self.request.fileno() .. with self.request.recv

365. By Gerdus van Zyl

-

364. By Gerdus van Zyl

disconnect cleanup, push to vpc

363. By Gerdus van Zyl

is_disconnection_error

362. By Gerdus van Zyl

test cases for returning and select limit

361. By Gerdus van Zyl

start of disconnect test, fix base database test to not use "select 1"

360. By Gerdus van Zyl

cleanup and override non implementable tests

359. By Gerdus van Zyl

code to pass test test_execute_expression and test_execute_expression_empty_params

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 status/vote changes: