[ 
https://issues.apache.org/jira/browse/WAVE-321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13187233#comment-13187233
 ] 

[email protected] commented on WAVE-321:
----------------------------------------------------



bq.  On 2012-01-16 21:55:27, Yuri Zelikov wrote:
bq.  > If it lets the server to continue with loading - it LGTM. But maybe need 
to open another issue to prevent this condition to happen.

Committed as r1232186. Thanks! Please close the review request.


- Yuri


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3432/#review4412
-----------------------------------------------------------


On 2012-01-16 21:46:20, Ali Lown wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/3432/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-01-16 21:46:20)
bq.  
bq.  
bq.  Review request for wave, Yuri Zelikov and Lennard de Rijk.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Nothing exciting in this patch, just prevents the length being processed 
as <0 (causing a whole server crash) for an uncaught exception.
bq.  
bq.  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.
bq.  
bq.  As for how it got corrupt I don't know so I can't address that actual 
issue.
bq.  
bq.  
bq.  This addresses bug WAVE-321.
bq.      https://issues.apache.org/jira/browse/WAVE-321
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    
/src/org/waveprotocol/box/server/persistence/file/FileDeltaCollection.java 
1230205 
bq.  
bq.  Diff: https://reviews.apache.org/r/3432/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  I wrote, compiled and deployed. Then my server booted again.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Ali
bq.  
bq.


                
> Exception during transforming deltas on start causes server to fail to start
> ----------------------------------------------------------------------------
>
>                 Key: WAVE-321
>                 URL: https://issues.apache.org/jira/browse/WAVE-321
>             Project: Wave
>          Issue Type: Bug
>          Components: Server
>         Environment: Debian sid.
>            Reporter: Ali Lown
>            Assignee: Ali Lown
>            Priority: Critical
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> One of my waves has a corrupt delta (somehow) which is preventing the server 
> from starting at all due to an exception not being handled.
> The stacktrace is:
> Exception in thread "main" java.lang.NegativeArraySizeException
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaCollection.readTransformedWaveletDelta(FileDeltaCollection.java:510)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaCollection.readTransformedDeltaFromRecord(FileDeltaCollection.java:435)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaCollection.access$100(FileDeltaCollection.java:64)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaCollection$1$1.hasNext(FileDeltaCollection.java:308)
>         at 
> org.waveprotocol.box.server.persistence.file.DeltaIndex.rebuildIndexFromDeltas(DeltaIndex.java:100)
>         at 
> org.waveprotocol.box.server.persistence.file.DeltaIndex.openForCollection(DeltaIndex.java:75)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaCollection.open(FileDeltaCollection.java:128)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaStore.open(FileDeltaStore.java:69)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaStore.lookup(FileDeltaStore.java:101)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaStore$3.fetchNext(FileDeltaStore.java:149)
>         at 
> org.waveprotocol.box.server.persistence.file.FileDeltaStore$3.hasNext(FileDeltaStore.java:161)
>         at 
> org.waveprotocol.box.server.waveserver.WaveMap.loadAllWavelets(WaveMap.java:682)
>         at 
> org.waveprotocol.box.server.waveserver.WaveServerImpl.initialize(WaveServerImpl.java:283)
>         at 
> org.waveprotocol.box.server.ServerMain.initializeServer(ServerMain.java:210)
>         at org.waveprotocol.box.server.ServerMain.run(ServerMain.java:174)
>         at org.waveprotocol.box.server.ServerMain.main(ServerMain.java:127)
> I am now working on a patch to enable the server to boot, simply ignoring the 
> corrupt delta.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to