Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(123)

Issue 11775018: Roll leveldb to r71 (Closed)

Created:
7 years, 11 months ago by dgrogan
Modified:
7 years, 11 months ago
Reviewers:
alecflett, jsbell
CC:
chromium-reviews
Visibility:
Public.

Description

Roll leveldb to r71. The only change applicable to chromium is a fix for a bug that causes corruption. The corruption manifests as a missing sst file that renders the database unopenable. BUG=168649 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175442

Patch Set 1 #

Patch Set 2 : update README.chromium #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M third_party/leveldatabase/README.chromium View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
dgrogan
Josh or Alec, could you review this?
7 years, 11 months ago (2013-01-07 22:20:06 UTC) #1
jsbell
On 2013/01/07 22:20:06, dgrogan wrote: > Josh or Alec, could you review this? Don't we ...
7 years, 11 months ago (2013-01-07 22:22:57 UTC) #2
dgrogan
On 2013/01/07 22:22:57, jsbell wrote: > On 2013/01/07 22:20:06, dgrogan wrote: > > Josh or ...
7 years, 11 months ago (2013-01-07 22:23:38 UTC) #3
jsbell
But the patch LGTM
7 years, 11 months ago (2013-01-07 22:24:07 UTC) #4
dgrogan
now includes README.chromium
7 years, 11 months ago (2013-01-07 22:25:06 UTC) #5
dgrogan
On 2013/01/07 22:25:06, dgrogan wrote: > now includes README.chromium Oh, I misread your message. More ...
7 years, 11 months ago (2013-01-07 22:25:43 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dgrogan@chromium.org/11775018/6001
7 years, 11 months ago (2013-01-07 22:32:54 UTC) #7
commit-bot: I haz the power
7 years, 11 months ago (2013-01-08 01:16:58 UTC) #8
Message was sent while issue was closed.
Change committed as 175442

Powered by Google App Engine
This is Rietveld 408576698