775
61%
main: 61%

Ran 11 Sep 2017 05:41AM UTC

Jobs 2

Files 21

Run time 2min

Badge

Embed ▾

pending completion

Build # 775

Build Type

push

travis-ci

Committed by

deepakjois

Commit Message

Use mmap for reading values everywhere.

This change adds support for mmap-ing the writable value log file,
in addition to the read-only files. We mmap the files with the size
set to math.MaxUint32, which is the theoretical maximum size of the value
pointer offset.

It is not very clear what the implications of mmap-ing beyond the
size of the file is. The mmap manpage does not indicate anything
specific. Moreover, it also says:

> “The effect of changing the size of the underlying file of a mapping
on the pages that correspond to added or removed regions of the
file is unspecified.”

The approach used here is similar to what BoltDB is doing, so there
is some confidence that it is fine.

Other notable changes:

* Since we are doing mmap reads exclusively, we no longer need to
  pass around a slice to copy the buffer. So some method signatures
  also changed as a result of that.

* Rename logFile.fdlock to logFile.lock because it is now used to guard
both file descriptor and the memory map.

* Add logFile.mmap() convenience method.

* Make logFile.readValueBytes() method take a callback for consuming
bytes.

Run Details

3147 of 4131 relevant lines covered (76.18%)

66284.8 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	775.1	11 Sep 2017 05:41AM UTC	21	76.18	Travis Job 775.1
2	775.2	11 Sep 2017 05:44AM UTC	21	75.96	Travis Job 775.2

Source Files on build 775

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 775

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 775

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

README BADGES
x