crash_test: add the new memtable to the parameters chosen #89
Assignees
Labels
enhancement
New feature or request
Comments
isaac-io
added a commit
that referenced
this issue
Jul 31, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
isaac-io
added a commit
that referenced
this issue
Jul 31, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
isaac-io
added a commit
that referenced
this issue
Aug 1, 2022
…h test The test seems to fail for now, so in order to avoid headaches for devs working on `main`, let's not test the new memtable until the issues are fixed.
isaac-io
added a commit
that referenced
this issue
Oct 19, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
isaac-io
added a commit
that referenced
this issue
Oct 19, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
isaac-io
added a commit
that referenced
this issue
Oct 19, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
Yuval-Ariel
pushed a commit
that referenced
this issue
Nov 23, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
Yuval-Ariel
pushed a commit
that referenced
this issue
Nov 25, 2022
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
Yuval-Ariel
pushed a commit
that referenced
this issue
Apr 30, 2023
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
udi-speedb
pushed a commit
that referenced
this issue
Oct 31, 2023
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
udi-speedb
pushed a commit
that referenced
this issue
Dec 1, 2023
Currently only the skip list memtable is being tested. Increase coverage by adding the new Speedb memtable as an option as well. For now we'll simply choose randomly between the two, without giving any one of them preference, but we may want to change that in the future.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently only the skip list is being tested as part of the crash tests (whitebox/blackbox). Add the new Speedb memtable to the available choices (for now just randomly choose between the skip list and the Speedb one) in order to cover it as part of the crash tests as well.
The text was updated successfully, but these errors were encountered: