Disable the "not expect lag" check from disorder ExecuteScript.

"Apparent" lag of 10 seconds can happen, the way confirm
messages propagate back to the origin.

diff --git a/clustertest/disorder/tests/ExecuteScript.js b/clustertest/disorder/tests/ExecuteScript.js
index 53def344a1ee1d4ec355f2b67d7344e72f9d71c7..d464fd683dc7b78f6009001422f792ceac41e246 100644 (file)
--- a/clustertest/disorder/tests/ExecuteScript.js
+++ b/clustertest/disorder/tests/ExecuteScript.js
@@ -437,9 +437,13 @@ ExecuteScript.prototype.testDDLFailure = function() {
         */
        statement.execute('DROP TABLE disorder.test_transient');
        this.slonikSync(1, 1);
-       var lag = this.measureLag(1,2);
-       this.coordinator.log('we do not expect lag, we measure it as ' + lag);
-       this.testResults.assertCheck('node is not lagged', lag < 10,true);
+       /**
+        * sl_status showing some lag of 10 seconds or more is not
+        * unusual because of the way confirms propagate.
+        */
+       // var lag = this.measureLag(1,2);
+       // this.coordinator.log('we do not expect lag, we measure it as ' + lag);
+       // this.testResults.assertCheck('node is not lagged', lag < 10,true);
        this.slonikSync(1,1);
        this.dropTestTable(1,1,false);