739
76%
main: 61%

Ran 07 Sep 2017 04:27AM UTC

Jobs 2

Files 22

Run time 72min

Badge

Embed ▾

pending completion

Build # 739

Build Type

push

travis-ci

Committed by

deepakjois

Commit Message

Make KVItem.Value() method asynchronous, and return error.

KVItem.Value() now takes a func that is called with the slice containing
the bytes for the value. This slice directly references the memory map
for the value log. It allows us to avoid copying bytes from the mmap-ed
value log.

Other related changes:

* Add a field to KVItem to hold a reference to the corresponding KV.

* Add a field to KVItem to track whether the value has been prefetched.

* Change the semantics of iteration. We don’t fetch values by design
anymore. So FetchValues does not apply normally. It only applies in the
case of prefetching. So renamed it to PrefetchValues. PrefetchSize now
only applies if PrefetchValues is true.

* If PrefetchValues is true, we copy values from mmap and set item.val.

* Remove KV.FillValue() method. We don’t fetch value synchronously
anymore, this method is not relevant.

* Refactor some private methods in value.go

* Fixes to tests. Added a helper method to asynchronously retrieve the
value and assign it to a variable.

Run Details

3194 of 4211 relevant lines covered (75.85%)

64892.64 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	739.1	07 Sep 2017 04:27AM UTC	22	75.8	Travis Job 739.1
2	739.2	07 Sep 2017 05:39AM UTC	22	75.85	Travis Job 739.2

Source Files on build 739

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

dgraph-io / badger / 739
76%
main: 61%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 739

dgraph-io / badger / 739 76% main: 61%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 739

dgraph-io / badger / 739
76%
main: 61%

README BADGES
x