[gmx-developers] new gerrit

[Szilárd Páll]

Nice indeed, I see quite a bit of performance improvement - especially
with large change-sets!

However, FYI I also (still) notice occasional lag on the interface,
e.g. when switching to the next file in a large change like 2704,
which tends to correlate with spikes on the server CPU usage, but I'm
not sure that the source of latency is indeed the CPU-bound nature of
some gerrit server operations.

Is it possible to make the interface a bit more compact, in particular:
- the in-source comment blocks take a lot of vertical space and
- the new-old change view - in collapsed mode every comment takes
about 2x more vertical space (I know the new view is more compact, but
I switched back from it for now because the new layout was rather
confusing).

Cheers,
--
Szilárd


On Mon, Jan 20, 2014 at 4:06 PM, Mark Abraham <mark.j.abraham at gmail.com> wrote:
> Thanks, Roland!
>
> The new change screen looks like a net improvement, though I have filed two
> minor bug reports for the new change screen (no line wrap with side-by-side
> diff, and non-keyboard intra-patch navigation is now harder for no
> particularly good reason).
>
> Mark
>
>
> On Fri, Jan 17, 2014 at 7:23 AM, Roland Schulz <roland at utk.edu> wrote:
>>
>> Hi,
>>
>> I just updated Gerrit.
>>
>> New release highlights:
>> - New change screen. Can be enabled at
>> https://gerrit.gromacs.org/#/settings/preferences per user. I think the new
>> one is nicer and I recommend switching. See
>> https://gerrit.gromacs.org/Documentation/intro-change-screen.html for a
>> quick introduction.
>> - Full text search for comments and file-name search. See:
>> https://gerrit-documentation.storage.googleapis.com/Documentation/2.8/user-search.html#comment
>> - Review votes are kept for new trivial changes (rebase + comment change)
>> - Reviewers are automatically added based on "git blame"
>>
>> For more details see:
>>
>> https://gerrit-documentation.storage.googleapis.com/ReleaseNotes/ReleaseNotes-2.8.html
>>
>> https://gerrit-documentation.storage.googleapis.com/ReleaseNotes/ReleaseNotes-2.7.html
>> We had 2.6 before. Thus both ReleaseNotes apply.
>>
>> Roland
>>
>> --
>> ORNL/UT Center for Molecular Biophysics cmb.ornl.gov
>> 865-241-1537, ORNL PO BOX 2008 MS6309
>>
>> --
>> Gromacs Developers mailing list
>>
>> * Please search the archive at
>> http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List before
>> posting!
>>
>> * Can't post? Read http://www.gromacs.org/Support/Mailing_Lists
>>
>> * For (un)subscribe requests visit
>> https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers or
>> send a mail to gmx-developers-request at gromacs.org.
>
>
>
> --
> Gromacs Developers mailing list
>
> * Please search the archive at
> http://www.gromacs.org/Support/Mailing_Lists/GMX-developers_List before
> posting!
>
> * Can't post? Read http://www.gromacs.org/Support/Mailing_Lists
>
> * For (un)subscribe requests visit
> https://maillist.sys.kth.se/mailman/listinfo/gromacs.org_gmx-developers or
> send a mail to gmx-developers-request at gromacs.org.