Code review comment for lp:~laurynas-biveinis/percona-server/fake-changes-from-fb-5.6

Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

thr_get_trx expects that thr is NULL

btr0cur.cc line 2397, btr_cur_optimistic_update

line 2260:
ut_ad(thr_get_trx(thr)->id == trx_id
      || (flags & ~(BTR_KEEP_POS_FLAG | BTR_KEEP_IBUF_BITMAP))
      == (BTR_NO_UNDO_LOG_FLAG | BTR_NO_LOCKING_FLAG
   | BTR_CREATE_FLAG | BTR_KEEP_SYS_FLAG));
is either bug or thr should never be NULL here

lock0lock.cc line 6162, lock_clust_rec_modify_check_and_lock

line 6177:
ut_ad(lock_table_has(thr_get_trx(thr), index->table, LOCK_IX));

lock0lock.cc line 6228, lock_sec_rec_modify_check_and_lock

line 6241:
ut_ad(lock_table_has(thr_get_trx(thr), index->table, LOCK_IX));

que0que.cc line 6228, que_eval_sql

line 1294:
graph->trx = trx;

« Back to merge proposal