search

scalar-labs / scalar-jepsen

Jepsen tests for ScalarDB and ScalarDL
45 stars 6 forks source link

Add elle-append test #49

Closed yito88 closed 3 years ago

yito88 commented 3 years ago

To verify serializable transactions: https://scalar-labs.atlassian.net/browse/DLT-6513

The current Scalar DB (2.4.0) using snapshot isolation (also in the case using serializable) failed in this test because a transaction could read a different value after another transaction committed. https://scalar-labs.atlassian.net/browse/DLT-7969

yito88 commented 3 years ago

@feeblefakie Thank you for reviewing. I added unit tests and fix minor stuff. Please take a look again.

Tests with jepsen.tests.cycle.wr are also coming?

Ah, I didn't intend to add a test with jepsen.tests.cycle.wr. However, I'd like to add the test now since I could make the test with append earlier than I expected.