Opened 12 years ago

Closed 11 years ago

#17574 closed Cleanup/optimization (fixed)

Cleanup in the introspection code

Reported by: Claude Paroz Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords: introspection
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

While working on #7783, I did some improvements on the introspection code:

  • moved the 'standard' get_primary_key_column at base backend (only SQLite overrides it)
  • added the missing get_key_columns implementation for PostgreSQL
  • in tests, the IgnoreNotimplementedError class should be replaced by @skip decorators if needed
  • test_get_key_columns should be skipped on MySQL with MyISAM engine

Tested with SQLite, MySQL (InnoDB and MyISAM), PostgreSQL. I don't have access to Oracle to test.

Attachments (2)

17574-1.diff​ (5.5 KB ) - added by Claude Paroz 12 years ago.
Introspection code improvements
17574-2.diff​ (3.8 KB ) - added by Claude Paroz 12 years ago.
Updated after 09a9971

Download all attachments as: .zip

Change History (6)

by Claude Paroz, 12 years ago

Attachment: 17574-1.diff​ added

Introspection code improvements

comment:1 by Jannis Leidel, 12 years ago

Triage Stage: Unreviewed → Accepted

comment:2 by Claude Paroz <claude@…>, 12 years ago

In [09a99714c05316717d2797afd2e458dbf6aa880f]:

Moved get_primary_key_column implementation to base

Refs #17574.

by Claude Paroz, 12 years ago

Attachment: 17574-2.diff​ added

Updated after 09a9971

comment:3 by Claude Paroz, 12 years ago

Still needs to be tested with Oracle

comment:4 by Claude Paroz <claude@…>, 11 years ago

Resolution: → fixed
Status: new → closed

In 0171ba65dbbff377282c03b86c83036168c84b22:

Fixed #17574 -- Implemented missing get_key_columns in PostgreSQL backend

Note: See TracTickets for help on using tickets.
Back to Top