New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 750817 link

Starred by 1 user
Status: Fixed
Owner:
cmumford@chromium.org
Closed: Sep 2017
Cc:
█ dskiba@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Feature

Blocking:
issue 750751



Sign in to add a comment

Ensure all users of leveldb are using DBTracker

Project Member Reported by cmumford@chromium.org, Jul 31 2017 
To ensure all leveldb use is reported to memory-infra verify that all leveldb activity uses the DBTracker.

Comment 1 by cmumford@chromium.org, Jul 31 2017

Labels: -Type-Task Type-Feature

Comment 2 by cmumford@chromium.org, Jul 31 2017

Blocking: 750751

Comment 3 by cmumford@chromium.org, Aug 3 2017

Cc: dskiba@chromium.org
Labels: Needs-Feedback
dskiba@ When you landed CL #488099 (https://codereview.chromium.org/2953473002) that switched all (on-disk and in-memory) leveldb's to using leveldb_env::OpenDB() which makes them tracked right? Which ones are missing?

Comment 4 by dskiba@chromium.org, Aug 7 2017 

Yes, all leveldb databases are reported by DBTracker. There is a presubmit check that should prevent us from adding new ones.

Comment 5 by cmumford@chromium.org, Sep 5 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment