Deprecate IDatabase::nextSequenceValue()
authorBrad Jorsch <bjorsch@wikimedia.org>
Thu, 31 Aug 2017 00:27:51 +0000 (20:27 -0400)
committerBrad Jorsch <bjorsch@wikimedia.org>
Thu, 31 Aug 2017 15:16:26 +0000 (11:16 -0400)
It's often forgotten because MySQL and Sqlite don't use it, the only
users are PostgreSQL and Oracle. And when used, if inserts to multiple
tables are being done it's easy to get the ordering wrong.

This patch reimplements DatabasePostgres::insertId() in terms of PG's
lastval() function, and adds triggers to the Oracle schema to make it
work the same as the other databases.

Bug: T164900
Change-Id: Ib308190c52673a9266c8495a589ae644f9fbefce


No differences found