deprecate inspector.get_primary_keys() ?

[Mike Bayer]

this method is redundant vs. get_pk_constraint() and also not named very well. All the dialects could just as well just implement get_pk_constraint() internally too. The method doesn't seem to be used by itself.

[Mike Bayer]

looks great. I think we want to leave inspector.get_primary_keys() in, mark with @deprecated for now, in case anyone is using it.

[Former user]

(original author: diana) Replying to zzzeek:

looks great. I think we want to leave inspector.get_primary_keys() in, mark with @deprecated for now, in case anyone is using it.

Ok. I'll add it back... Friday night (busy until then). And then re-submit. Thx!

[Former user]

(original author: diana)

[Former user]

(original author: diana) Toying with pull requests instead of patches.

https://bitbucket.org/sqlalchemy/sqlalchemy/pull-request/3/deprecate-inspectorget_primary_keys-in

[Mike Bayer]

1fe06a551c28a6e0886f96334deebdee68d9fff9 in the 0.8 branch

[Mike Bayer]

Removing milestone: 0.8.0b1 (automated comment)