----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3432/#review4412 -----------------------------------------------------------
Ship it! If it lets the server to continue with loading - it LGTM. But maybe need to open another issue to prevent this condition to happen. - Yuri On 2012-01-16 21:46:20, Ali Lown wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3432/ > ----------------------------------------------------------- > > (Updated 2012-01-16 21:46:20) > > > Review request for wave, Yuri Zelikov and Lennard de Rijk. > > > Summary > ------- > > Nothing exciting in this patch, just prevents the length being processed as > <0 (causing a whole server crash) for an uncaught exception. > > This lets the server boot (which is much more important that a few corrupt > deltas), and the rest of the persistence layer still leaves a SEVERE in my > logs to tell me about the corrupt delta in the wave. > > As for how it got corrupt I don't know so I can't address that actual issue. > > > This addresses bug WAVE-321. > https://issues.apache.org/jira/browse/WAVE-321 > > > Diffs > ----- > > /src/org/waveprotocol/box/server/persistence/file/FileDeltaCollection.java > 1230205 > > Diff: https://reviews.apache.org/r/3432/diff > > > Testing > ------- > > I wrote, compiled and deployed. Then my server booted again. > > > Thanks, > > Ali > >
