bzr commit into mysql-5.1-telco-6.2 tree (frazer:2633) Bug#37672

[Frazer Clement]

#At file:///home/frazer/bzr/mysql-5.1-telco-6.2/

------------------------------------------------------------
revno: 2633
revision-id: frazer@stripped
parent: jonas@stripped
committer: Frazer Clement <frazer@stripped>
branch nick: mysql-5.1-telco-6.2
timestamp: Fri 2008-07-04 13:37:19 +0100
message:
  Bug#37672 NDBAPI : NdbRecord option OO_ANYVALUE causes interpreted delete to abort.
  
  Setting the ANYVALUE operation option for an NdbRecord interpreted delete 
  results in transaction abort.  Error 888 is given.
  
  2 problems : 
   1) For delete, NdbRecord code was placing ANYVALUE update ATTRINFO in the 
      'final Update' part of the TCKEYREQ rather than the 'initial Read' 
      part.
   2) For interpreted delete, TUP was ignoring log ATTRINFO words generated 
      by read part of interpreted delete, resulting in SUMA sending events
      without correct ANYVALUE.
modified:
  storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp
sp1f-dbtupexecquery.cpp-20040414082419-mq6uenb66nxl6kcdp3v27evwjkcs2ttz
  storage/ndb/src/ndbapi/NdbOperationExec.cpp
sp1f-ndboperationexec.cpp-20040414082425-spfhlvqpx5hh2u7nyud2l5ordg7u43eb
  storage/ndb/test/ndbapi/test_event.cpp
sp1f-test_event.cpp-20040414082438-mn2bippetrmdotxdqygnmo7y2fhcioia
  storage/ndb/test/run-test/daily-basic-tests.txt
sp1f-dailybasictests.txt-20040623115449-liur6p3tedydxdte6rr35pqap26frov6
per-file comments:
  storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp
    Modify TUP interpreter to include log words generated as a side-effect of
interpretation.
  storage/ndb/src/ndbapi/NdbOperationExec.cpp
    Modify NdbRecord TCKEYREQ generating code to put ANYVALUE ATTRINFO in initial read
section for delete requests.
  storage/ndb/test/ndbapi/test_event.cpp
    Add testcase to check that AnyValue is correctly propagated to events for PK insert,
update, write (update), delete with and without interpreted code.
  storage/ndb/test/run-test/daily-basic-tests.txt
    Add new testcase to daily-basic.

Attachment: [text/text/x-diff] patch-2633.diff