TransactionalStorageEngine::doRollback(session, all) "all" isn't true - need to check session autocommit settings

Bug #701468 reported by Stewart Smith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Drizzle
Fix Released
Medium
Stewart Smith
7.0
Fix Released
Medium
Stewart Smith

Bug Description

Very much related a bit to: https://bugs.launchpad.net/drizzle/+bug/682563

basically, the same for rollback:

int SEAPITester::doRollback(Session *session, bool all)
{
  if (all
    || !session_test_options(session, OPTION_NOT_AUTOCOMMIT | OPTION_BEGIN))
  {
    ENGINE_NEW_STATE("ROLLBACK");
    ENGINE_NEW_STATE("::SEAPITester()");
  }
  else
  {
    ENGINE_NEW_STATE("ROLLBACK STATEMENT");
    ENGINE_NEW_STATE("In Transaction");
  }

  return getRealEngine()->rollback(session, all);
}

Related branches

Stewart Smith (stewart)
Changed in drizzle:
status: In Progress → Fix Committed
Stewart Smith (stewart)
Changed in drizzle:
milestone: 2011-01-17 → 2011-01-31
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.