DescriptionRevert of Add LevelDB database names to tracing path for easier debugging (patchset #2 id:40001 of https://codereview.chromium.org/2399903004/ )
Reason for revert:
We identified the largest consumer of memory in leveldb_proto databases and fixed it crbug.com/662019.
It is not safe to add dumps with database names since new clients of leveldb_proto might not update the whitelist and we will miss the memory usage.
Original issue's description:
> Add LevelDB database names to tracing path for easier debugging
>
> The data from field trials show large databases, but not sure what are
> their names. So, adding names to the path.
>
> BUG=645126
>
> Committed: https://crrev.com/279f8d702e8b2fffda1b7f50b311c5202331e121
> Cr-Commit-Position: refs/heads/master@{#424055}
TBR=primiano@chromium.org,nyquist@chromium.org,mathp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=645126
Review-Url: https://codereview.chromium.org/2601233002
Cr-Commit-Position: refs/heads/master@{#442782}
Committed: https://chromium.googlesource.com/chromium/src/+/c5f7441b83ffcb113514512a65ca5910a9b5115b
Patch Set 1 #
Messages
Total messages: 9 (3 generated)
|