Re: Issue 2788 in reviewboard: after update some old reviews fail to show diff view.

2012-10-29 Thread reviewboard


Comment #4 on issue 2788 by chip...@gmail.com: after update some old  
reviews fail to show diff view.

http://code.google.com/p/reviewboard/issues/detail?id=2788

It's a bug. I'd rather fix it than have setup tell you to work around it :)  
(And if we knew about it before releasing, we'd have fixed it then.)


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2788 in reviewboard: after update some old reviews fail to show diff view.

2012-10-29 Thread reviewboard


Comment #3 on issue 2788 by dotheb...@gmail.com: after update some old  
reviews fail to show diff view.

http://code.google.com/p/reviewboard/issues/detail?id=2788

yes, this fixes the behaviour.
Maybe setup should point the user to restart memcached in such a case.

Tnx a lot.

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2788 in reviewboard: after update some old reviews fail to show diff view.

2012-10-29 Thread reviewboard

Updates:
Status: Confirmed
Owner: chip...@gmail.com
	Labels: -Priority-Medium Priority-High Milestone-Release1.6.x  
Component-DiffViewer


Comment #2 on issue 2788 by chip...@gmail.com: after update some old  
reviews fail to show diff view.

http://code.google.com/p/reviewboard/issues/detail?id=2788

Restart your memcached and the problem will go away. A cache key didn't get  
bumped when the format of the diff cache changed. We'll deploy a fix in the  
next release.


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2788 in reviewboard: after update some old reviews fail to show diff view.

2012-10-29 Thread reviewboard


Comment #1 on issue 2788 by dotheb...@gmail.com: after update some old  
reviews fail to show diff view.

http://code.google.com/p/reviewboard/issues/detail?id=2788

behaviour also reproduceable occurs i.e. in partial diffs:
/r/373/diff/2/fragment/6386/?index=6&1351159979

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.