1613
61%
main: 61%

Ran 25 Sep 2018 04:19AM CUT

Jobs 1

Files 27

Run time 8s

Badge

Embed ▾

Committed 25 Sep 2018 04:13AM CUT coverage: 80.396% (-0.4%) from 80.769%

Build # 1613

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Add a Jepsen style bank test for Badger (#577)

**Breaking Change**

This change breaks Value API. Instead of Value returning a byte slice, we now expose the byte slice from within a function. This is because when we read a value, we acquire a read lock on the value log. With the previous API it was possible for a transaction to deadlock, if multiple reads are done on the same log file, and those read locks get interleaved with a write lock caused by log file rotation. The new API fixes that by releasing the lock before another value is read; and also makes it obvious to the end user that they must copy the value to be able to use it outside the function.

This change also simplifies the transaction timestamp logic, learning from Dgraph. It simplifies it down to use just one variable `nextTxnTs` for tracking timestamps.

This change also adds a Jepsen style bank test for Badger, which is currently showing some transaction violations, needing further investigation.

Run Details

4470 of 5560 relevant lines covered (80.4%)

105238.36 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	1613.1	25 Sep 2018 04:19AM CUT	0	80.4	Travis Job 1613.1

dgraph-io / badger / 1613
61%
main: 61%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1613

dgraph-io / badger / 1613 61% main: 61%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1613

dgraph-io / badger / 1613
61%
main: 61%

README BADGES
x