----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3815/#review12722 -----------------------------------------------------------
Ship it! Ship It! - Mark Michelson On July 17, 2014, 5:57 p.m., Michael Young wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3815/ > ----------------------------------------------------------- > > (Updated July 17, 2014, 5:57 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24050 > https://issues.asterisk.org/jira/browse/ASTERISK-24050 > > > Repository: Asterisk > > > Description > ------- > > When updating a row, we are currently doing an INSERT OR REPLACE INTO. The > downside to this is that the row is deleted if it exists and then a new row > is inserted. So, we are hitting the disk twice. One for the deletion and > one for the insertion. > > The proposed patch will attempt to do an INSERT INTO and if it fails because > a row with that key exists, we will ignore that. Then we will attempt to > perform an UPDATE on the existing row. If a record was INSERTED, the UPDATE > statement will end up doing nothing. > > > Diffs > ----- > > /trunk/main/db.c 418608 > /trunk/include/asterisk/astdb.h 418608 > > Diff: https://reviewboard.asterisk.org/r/3815/diff/ > > > Testing > ------- > > Local pbx with a couple of peers. Did see a slight I/O increase. > > before patch > Re-Register Avg. 404 ms > Register Avg. 442 ms > > after patch > Re-Register Avg. 361.5 ms > Register Avg. 419 ms > > > Thanks, > > Michael Young > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
