From d9cb0cd6e35849e041d94f5ec175e5b0cca24f78 Mon Sep 17 00:00:00 2001
From: Jan Christoph Uhde <ObiWahn@users.noreply.github.com>
Date: Thu, 28 Nov 2019 14:55:41 +0100
Subject: [PATCH] Bug fix/test shell skiplist rm performance (#10557)

* de-graylist shell-skiplist-rm-performance-timecritical-noncluster

* increase delta

* delete test
---
 ...erformance-timecritical-noncluster-grey.js | 130 ------------------
 1 file changed, 130 deletions(-)
 delete mode 100644 tests/js/server/shell/shell-skiplist-rm-performance-timecritical-noncluster-grey.js

diff --git a/tests/js/server/shell/shell-skiplist-rm-performance-timecritical-noncluster-grey.js b/tests/js/server/shell/shell-skiplist-rm-performance-timecritical-noncluster-grey.js
deleted file mode 100644
index 70fc1aa33b..0000000000
--- a/tests/js/server/shell/shell-skiplist-rm-performance-timecritical-noncluster-grey.js
+++ /dev/null
@@ -1,130 +0,0 @@
-/*jshint globalstrict:false, strict:false */
-/*global assertFalse */
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief test the performance of removal with a skip-list index
-///
-/// @file
-///
-/// DISCLAIMER
-///
-/// Copyright 2013 triagens GmbH, Cologne, Germany
-///
-/// Licensed under the Apache License, Version 2.0 (the "License");
-/// you may not use this file except in compliance with the License.
-/// You may obtain a copy of the License at
-///
-///     http://www.apache.org/licenses/LICENSE-2.0
-///
-/// Unless required by applicable law or agreed to in writing, software
-/// distributed under the License is distributed on an "AS IS" BASIS,
-/// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-/// See the License for the specific language governing permissions and
-/// limitations under the License.
-///
-/// Copyright holder is triAGENS GmbH, Cologne, Germany
-///
-/// @author Max Neunhoeffer
-/// @author Copyright 2013, triAGENS GmbH, Cologne, Germany
-////////////////////////////////////////////////////////////////////////////////
-
-var jsunity = require("jsunity");
-var internal = require("internal");
-var db = require("internal").db;
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief test suite: Creation
-////////////////////////////////////////////////////////////////////////////////
-
-function SkipListPerfSuite() {
-  'use strict';
-  var cn = "UnitTestsCollectionSkiplistPerf";
-  var collection = null;
-
-  return {
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief set up
-////////////////////////////////////////////////////////////////////////////////
-
-  setUp : function () {
-    internal.db._drop(cn);
-    collection = internal.db._create(cn, { waitForSync : false });
-  },
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief tear down
-////////////////////////////////////////////////////////////////////////////////
-
-  tearDown : function () {
-    // try...catch is necessary as some tests delete the collection itself!
-    try {
-      collection.unload();
-      collection.drop();
-    }
-    catch (err) {
-    }
-
-    collection = null;
-    internal.wait(0.0);
-  },
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief test: performance of deletion with skip-list index
-////////////////////////////////////////////////////////////////////////////////
-
-    testDeletionPerformance : function () {
-      var time = require("internal").time;
-      collection.ensureSkiplist("value");
-      var N=100000;
-      var p=14777;  // must be coprime to N
-
-      if (db._engine().name === "rocksdb") {
-        N = 1000;
-        p = 333;
-      }
-
-      for (i = 0;i < N;i++) {
-        collection.save({value:i});
-      }
-      var l = collection.toArray();
-      var t = time();
-      var j = 0;
-      var x;
-      for (var i = 0;i < l.length;i++) {
-        x = l[j];
-        j = (j+p) % l.length;
-        collection.remove(x._key);
-      }
-      var t1 = time()-t;
-      internal.db._drop(cn);
-      collection = internal.db._create(cn);
-      collection.ensureSkiplist("value");
-      for (i = 0;i < N;i++) {
-        collection.save({value: i % 10});
-      }
-      l = collection.toArray();
-      t = time();
-      j = 0;
-      for (i = 0;i < l.length;i++) {
-        x = l[j];
-        j = (j+p) % l.length;
-        collection.remove(x._key);
-      }
-      var t2 = time()-t;
-      assertFalse(t2 > 5*t1,"Removal with skip-list index is slow");
-    }
-
-  };
-}
-
-
-////////////////////////////////////////////////////////////////////////////////
-/// @brief executes the test suites
-////////////////////////////////////////////////////////////////////////////////
-
-jsunity.run(SkipListPerfSuite);
-
-return jsunity.done();
-
-