Reduce footprint of EliasFanoReader
authorGiuseppe Ottaviano <ott@fb.com>
Thu, 4 Jun 2015 04:31:16 +0000 (21:31 -0700)
committerNoam Lerner <noamler@fb.com>
Fri, 5 Jun 2015 02:32:09 +0000 (19:32 -0700)
commitf85b4b76bbc5cd9195bfa36ad60dd6e0a3c9cdb1
treee053adfa46e1e945a592aaf719e8f90adaced30e
parenta3537a0f553e89ca911745fa2e58db454044fb6b
Reduce footprint of EliasFanoReader

Summary:
`EliasFanoReader` has a copy of `EliasFanoCompressedList` as member, but it only needs few of its members. With this diff, it only copies the members it needs.
Also, `progress_` is a duplicate of `upper_.position()`, so it was removed.
Microbenchmarks do not indicate any significant change in performance.

Test Plan: unit tests

Reviewed By: philipp@fb.com

Subscribers: chaoyc, search-fbcode-diffs@, unicorn-diffs@, folly-diffs@, yfeldblum, tudort, chalfant

FB internal diff: D2125956

Signature: t1:2125956:1433381848:2a333ce7a741bec5d059e9e771309463d6018ea2
folly/experimental/EliasFanoCoding.h