MyRocks - gap lock error logic does not account for ISO mode

[George Ormond Lorch III]

https://jira.percona.com/browse/MYR-35

Our gap lock error implementation (from MYR-15) currently is not taking into account the current ISO mode. This logic should be changed to amend an additional condition before deciding to error to the client:
{code:java}
&& thd_tx_isolation(thd) >= ISO_REPEATABLE_READ
{code}
Therefore, if the ISO is <= READ COMMITTED, the gap lock error will not be raised in the condition of a gap lock, but as the ISO mode moves any further north in the consistency scale to REPEATABLE READ or SERIALIZABLE, then the gap lock error will be raised when conditions are met.

We will need to clearly document this behavior and the suggested default ISO mode of READ COMMITTED with MyRocks use cases. We will also need to update the MyRocks mtr suites to explicitly test ISO modes against the gap lock error, change the default ISO mode for the suites to READ COMMITTED, and re-word the error message.

[George Ormond Lorch III (gl-az)]

5.6:
http://jenkins.percona.com/view/PS%205.6/job/percona-server-5.6-param/1606/

5.7:
http://jenkins.percona.com/view/5.7/job/mysql-5.7-param/593/

[George Ormond Lorch III (gl-az)]

5.6:
https://github.com/percona/percona-server/pull/1311

5.7:
https://github.com/percona/percona-server/pull/1312

[Shahriyar Rzayev (rzayev-sehriyar)]

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PS-1776