When you hit this problem with large numbers of comments, how many reviews
are they usually spread across? Solving the problem of large numbers of
comments across multiple reviews is very different from large numbers of
comments on one review.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com


On Fri, Mar 6, 2009 at 2:54 PM, Christian Hammond <chip...@chipx86.com>wrote:

> We'd have to decide what we're doing to fix this first. Depending on what
> that is, it could take a few days to implement, or longer. We can make it a
> priority for beta 1 (the next release), and of course you'd be able to just
> upgrade to a nightly once it's in.
>
> Short-term, I'd just advise splitting up the changes more, if possible.
> Having smaller things to review should mean fewer comments.
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.review-board.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Fri, Mar 6, 2009 at 2:51 PM, mary <ciaom...@gmail.com> wrote:
>
>>
>> Yes, I mean comments not reviews.
>>
>> I'm seeing the same issues on Alpha4 on my test server. It seems to me
>> that the loading of the diff fragments across all the comments is
>> causing our problems - loading such a review sometimes crashes the
>> browser (i've seen this on firefox mostly) and in IE the page often
>> shows a script error popup box part way through the load.
>>
>> Changing the page size would help us so much, can you give any
>> indication of a time frame for such a change?
>>
>> We'd benefit from the other suggestions as well, but jsut getting
>> something workable for medium-to-large reviews is our immediate
>> concern.
>>
>> Thanks!
>>
>> On Mar 6, 2:39 pm, Christian Hammond <chip...@chipx86.com> wrote:
>> > I think we should. The thing is that the newest review request is at the
>> > bottom, so it's kinda weird.
>> >
>> > Another thing we should look into is auto-collapsing old reviews (such
>> as
>> > reviews made before the last update to the review request), allowing
>> them to
>> > expand again. This would fetch the collapsed items from the server
>> > dynamically.
>> >
>> > Scalability of the review request page is certainly something we should
>> > tackle for 1.0.
>> >
>> > As far as using Alpha 2 vs. Alpha 4, if you use Alpha 4 the page should
>> load
>> > pretty fast, with the diff fragments loading dynamically after. Even
>> with
>> > 150+ comments (do you mean actual comments or reviews, btw?) it
>> shouldn't
>> > take forever in alpha 4 to display those. Just might take a while for
>> those
>> > diff fragments ot finish loading across all comments.
>> >
>> > Christian
>> >
>> > --
>> > Christian Hammond - chip...@chipx86.com
>> > Review Board -http://www.review-board.org
>> > VMware, Inc. -http://www.vmware.com
>> >
>> > On Fri, Mar 6, 2009 at 2:06 PM, David Trowbridge <trowb...@gmail.com>
>> wrote:
>> >
>> > > Perhaps we need to paginate the reviews page in addition to the diff?
>> >
>> > > -David
>> >
>> > > On Fri, Mar 6, 2009 at 2:03 PM, mary <ciaom...@gmail.com> wrote:
>> >
>> > > > Thank you for your reply. The slowness is:
>> > > > 1. page load takes several minutes (the more comments, the longer it
>> > > > takes)
>> > > > 2. typing a comment is very slow on reviews with many comments.
>> > > > 3. Scrolling on the review page is painful when many reviews
>> > > > Developers are speculating its due to a huge DOM and say that
>> > > > performance benchmarks seem relative to the document size,
>> complexity,
>> > > > and browser type (Safari works best, then FireFox, then IE.)
>> >
>> > > > Can we change the paging size the ReviewBoard uses? That would help
>> us
>> > > > most likely.
>> >
>> > > > Further details:
>> > > > yes, we're using memcache. 4G ram.
>> >
>> > > > We've been running Alpha2 the past couple weeks. But it seems
>> Alpha1-4
>> > > > also have same issues.
>> >
>> > > > On Mar 6, 1:42 pm, Christian Hammond <chip...@chipx86.com> wrote:
>> > > >> It's important to find out what's causing the slowdown. What part
>> is
>> > > being
>> > > >> slow? The page itself, or the progressive diffs inside of it? Alpha
>> 1
>> > > >> doesn't have progressive diffs so it will be slower than alpha 4 in
>> this
>> > > >> regard.
>> >
>> > > >> We have some large review requests like this at VMware too, and
>> haven't
>> > > seen
>> > > >> this slowdown.
>> >
>> > > >> Are you using memcached on the server?
>> >
>> > > >> How much ram do you have on the server?
>> >
>> > > >> Christian
>> >
>> > > >> --
>> > > >> Christian Hammond - chip...@chipx86.com
>> > > >> Review Board -http://www.review-board.org
>> > > >> VMware, Inc. -http://www.vmware.com
>> >
>> > > >> On Fri, Mar 6, 2009 at 10:53 AM, mary <ciaom...@gmail.com> wrote:
>> >
>> > > >> > I was able to roll back to Alpha1 but ReviewBoard displayed the
>> same
>> > > >> > problem... an unresponsive/seriously slow GUI for reviews with
>> 150+
>> > > >> > comments. I also upgrade to Alpha4 with the same results.
>> >
>> > > >> > Our company cannot use ReviewBoard with this serious performance
>> > > >> > handicap.
>> >
>> > > >> > We've been very happy with ReviewBoard up until now. Is anyone
>> looking
>> > > >> > into this perf issue?
>> >
>> > > >> > We're going to have to move back to using <cringe> CodeStriker
>> soon if
>> > > >> > this isn't addressed.
>> >
>> > > >> > On Feb 24, 1:54 pm, mary <ciaom...@gmail.com> wrote:
>> > > >> > > Thanks for the information, I will try it on my test server
>> (and
>> > > yes,
>> > > >> > > I will keep in mind DB changes for future).
>> >
>> > > >> > > I've also been having a hard time reproducing the issues that
>> many
>> > > are
>> > > >> > > reporting but I will continue to try and gather information.
>> The
>> > > >> > > problems are being reported using both IE7 and Firefox 3 (both
>> from
>> > > >> > > Windows and Linux). Everyone is reporting that using Firefox is
>> > > >> > > definitely better than IE7 though, but still very slow. I have
>> > > >> > > reported one bug #906 which I thought was the full issue, but
>> others
>> > > >> > > are reporting the slowness even on reviews with a small diff
>> and
>> > > very
>> > > >> > > few total reviews.
>> > > >> >http://code.google.com/p/reviewboard/issues/detail?id=906
>> >
>> > > >> > > It's been reported numerous times that when users open up the
>> review
>> > > >> > > comment pop up box and start to type their comments, the
>> characters
>> > > >> > > typed take forever to show up in the comment field GUI. This is
>> new
>> > > >> > > for us, no one reported this prior to recent alpha upgrades (I
>> think
>> > > >> > > limited to alpha2, although i'm not certain of this.)
>> >
>> > > >> > > On Feb 24, 1:39 pm, Christian Hammond <chip...@chipx86.com>
>> wrote:
>> >
>> > > >> > > > In theory, you should be able to just install the  1.0alpha1
>> eggs
>> > > of
>> > > >> > both
>> > > >> > > > Review Board and Djblets. This is completely untested and
>> > > unsupported,
>> > > >> > > > though. Note that in the future, you'll have a harder time
>> with
>> > > this.
>> > > >> > There
>> > > >> > > > were no database schema changes between alpha 1 and 4 (to my
>> > > >> > knowledge), but
>> > > >> > > > there may be between 4 and some other version.
>> >
>> > > >> > > > I'm going to be committing a fix for interdiffs within a day
>> or
>> > > two.
>> > > >> > You
>> > > >> > > > could wait until then and upgrade to the nightly (which will
>> be
>> > > safe
>> > > >> > enough
>> > > >> > > > for use, as not much has changed since alpha 4).
>> >
>> > > >> > > > I don't know what this slow popup dialog issue is. I've heard
>> one
>> > > other
>> > > >> > > > person mention this but I can't reproduce it. The only way
>> it'll
>> > > get
>> > > >> > fixed,
>> > > >> > > > though, is if we can gather some debug info and figure out
>> what's
>> > > >> > causing
>> > > >> > > > it. Can you tell me what versions of what browsers on what
>> > > platforms
>> > > >> > they're
>> > > >> > > > using?
>> >
>> > > >> > > > Christian
>> >
>> > > >> > > > --
>> > > >> > > > Christian Hammond - chip...@chipx86.com
>> > > >> > > > Review Board -http://www.review-board.org
>> > > >> > > > VMware, Inc. -http://www.vmware.com
>> >
>> > > >> > > > On Tue, Feb 24, 2009 at 1:31 PM, mary <ciaom...@gmail.com>
>> wrote:
>> >
>> > > >> > > > > Hi,
>> >
>> > > >> > > > > Since upgrading to Alpha2 many of our users have been
>> reporting
>> > > >> > > > > seriously slow response times sporadically on the review
>> pages.
>> > > This
>> > > >> > > > > is making the GUI nearly unusable for some users. One page
>> > > >> > > > > specifically is the popup Review comment box. This does not
>> > > appear to
>> > > >> > > > > be network related and so I would like to roll back to
>> Alpha 1
>> > > to see
>> > > >> > > > > if that fixes the issue.
>> >
>> > > >> > > > > How do I roll back to a previous Alpha release? Can I
>> specify an
>> > > >> > alpha
>> > > >> > > > > version on install and just reinstall? It's not clear how
>> to do
>> > > this
>> > > >> > > > > from the docs.
>> >
>> > > >> > > > > Also, any estimate on when Alpha 5 will be ready? (We don't
>> want
>> > > to
>> > > >> > > > > upgrade to Alpha4 because of bug reported w/ viewing diffs
>> of
>> > > >> > > > > revisions, which we already have problems with.)
>> >
>> > > >> > > > > Thanks!
>> > > >> > > > > Mary- Hide quoted text -
>> >
>> > > >> > > - Show quoted text -- Hide quoted text -
>> >
>> > > >> - Show quoted text -
>> >>
>>
>

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

Reply via email to