Bug #35923 cross-thread EXECUTE_DIRECT in multi-threaded ndbd
Submitted: 9 Apr 2008 6:14 Modified: 15 Apr 2008 5:23
Reporter: Kristian Nielsen Email Updates:
Status: Closed Impact on me:
None 
Category:MySQL Cluster: Cluster (NDB) storage engine Severity:S3 (Non-critical)
Version:mysql-5.1-telco-6.4 OS:Any
Assigned to: Kristian Nielsen CPU Architecture:Any

[9 Apr 2008 6:14] Kristian Nielsen 
Description:
In DbtupBuffer.cpp, we send TRANSID_AI to TC block on local node using
EXECUTE_DIRECT(). This is wrong for multi-threaded ndbd, as it is not
threadsafe (TC and TUP run in different threads).

How to repeat:
Source inspection; also there is a test case in mysql-6.0-telco mysql-test-run.pl
(using batched join) that fails due to this.

Suggested fix:
Will commit patch that uses long signal send instead for this case.
[9 Apr 2008 6:16] Bugs System 
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/45103

ChangeSet@1.2595, 2008-04-09 08:15:37+02:00, knielsen@ymer.(none) +2 -0
  Bug #35923: cross-thread EXECUTE_DIRECT in multi-threaded ndbd
  
  Replace cross-thread EXECUTE_DIRECT with long send signal.
[9 Apr 2008 6:18] Bugs System 
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/45104

ChangeSet@1.2596, 2008-04-09 08:17:54+02:00, knielsen@ymer.(none) +1 -0
  Bug #35923: cross-thread EXECUTE_DIRECT in multi-threaded ndbd
  
  Indentation/whitespace only changes committed separately from real patch.
[9 Apr 2008 14:51] Kristian Nielsen 
Pushed to mysql-5.1-telco-6.4.

For documentation:

The bug is that there was a race condition in query execution inside the NDB
kernel for multithreaded ndb, which could cause transaction failure or crash.

But I am not sure if we need to document bugfixes for multi-threaded ndbd at this
stage, if not just close it.
[15 Apr 2008 5:23] Jon Stephens 
Doesn't appear in any release version, so no need to document. Closed.