• Kristian Nielsen's avatar
    Fix two races in test main.processlist that could cause random failures (seen in Buildbot) · 64af1ecc
    Kristian Nielsen authored
    1. Do not use NULL `info' field in processlist to select the thread of
    interest. This can fail if the read of processlist ends up happening after
    REAP succeeds, but before the `info' field is reset. Instead, select on the
    CONNECTION_ID(), making sure we still scan the whole list to trigger the same
    code as in the original test case.
    
    2. Wait for the query to really complete before reading it in the
    processlist. When REAP returns, it only means that ack has been sent to
    client, the reset of query stage happens a bit later in the code.
    64af1ecc
processlist.test 1.27 KB