Summary: Gerrit Hackathon at SAP 2016

[Saša Živkov]

Last week, from 19th until 23rd of September, there was a five days

Gerrit Hackathon at SAP in Walldorf.

The following people took part: Christian Halstrick, Dariusz Luksza,

Dave Borowitz, David Ostrovsky, David Pursehouse, Edwin Kempin,

Gustaf Lundh, Han-Wen Nienhuys, Hugo Arès, Jacek Centkowski,

Khai Do Khai, Matthias Sohn, Michael Ochmann, Patrick Hiesel,

Saša Živkov and Sven Selberg

I would like to thank to all 16 participants for their dedication and work!

It was also a great opportunity to get to know each other.

We put a hashtag "sap16" on all changes created or worked on during

the hackathon to make it easy to track [1] what's going on during the hackathon

and to also make it easier to write this summary :-)

Here is a detailed list of topics we worked on:

* Change Assignee workflow.

Introduce a new field on change: Assignee.

By definition, the assignee is responsible for a next step with that change (review, rebase, etc..)

The idea behind this new feature is to better handle scenarios where there are multiple reviewers

assigned on a change but no one feels responsible to take the next step.

* Change (re)indexing performance

For large Gerrit sites it could take a significant amount of time to reindex changes after an index schema

upgrade. Most of the time, however, reindexing computes diff in order to find

the list of changed files and size of the change. To address this issue we introduced

a new persistent cache: diff_summary. The purpose of this cache is to store everything

that is needed for change (re)indexing so that it doesn't need to be (re)computed again.

* ElasticSearch integration

Possibility to use ElasticSearch service instead of the embedded Lucene index.

There are still some TODOs like index schema upgrade and reindexing.

* New releases:

- JGit 4.5 released

- Gerrit 2.13 and 2.13.1 released

- Gerrit 2.12.5 released

* Code search UI in gitiles

The zoekt code-search [3] exposes a REST end point for performing queries

on indexed source code. Gitiles exposes UI for code-search [4] and utilizes the REST

end point from zoekt.

* Recommender for Reviewer Suggestion

Provide a recommender for suggesting change reviewer(s) [5].

Also provide an extension point so that plugin can provide own recommenders

for reviewer suggestion.

* Robot comments [6]: provide native support in Gerrit for Robot comments.

Some refactorings done during the hackathon and a first change uploaded yesterday [2]

* Switched to Java 8:

Example change: https://gerrit-review.googlesource.com/86360 shows

how utilizing streams and lambda functions can reduce some boilerplate code.

* Merge Commit review: for merge commits add a new magic file /MERGE_LIST

which provides a list of all commits merged into the target branch via this merge.

It is possiblet o post comments on the /MERGE_LIST.

* Polygerrit UI switch

Add a possibility to switch between the GWT and PolyGerrit UI from the UI itself.

* JGit: implemented built-in clean and smudge filters to support LFS protocol.

The smudge filter is not yet fully finished as it currently cannot download missing large object(s).

* delete-project plugin: fix project (re)creation after its deletion, the fix is actually done in JGit

and is included in JGit 4.5 release.

* WIP: integrating JGit Ketch into Gerrit

* Build:

- Bazel build for many plugins

- Basel build for GWT UI

- Work in progress Bazel build for PolyGerrit

- Upgrade GWT to 2.8-rc2

- Upgrade Buck to the latest version

[1] https://gerrit-review.googlesource.com/#/q/hashtag:sap16

[2] https://gerrit-review.googlesource.com/#/c/87093/

[3] https://gerrit.googlesource.com/zoekt/+/refs/heads/master

[4] https://gerrit-review.googlesource.com/86308

[5] https://gerrit-review.googlesource.com/86752

[6] https://docs.google.com/document/d/1pLunr0YUvbpFVjdgalEM0IX-Cm0EbpLxfDP1QryGRXM/edit

[luca.mi...@gmail.com]

Well done mates, a lot of outstanding achievements !

Luca

Sent from my iPhone

--
--
To unsubscribe, email repo-discuss...@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Sébastien Douche]

On Tue, Sep 27, 2016, at 17:11, Saša Živkov wrote:
>
> I would like to thank to all 16 participants for their dedication and
> work!

Well done!



--
Sébastien Douche <s...@nmeos.net>
Twitter: @sdouche
http://douche.name

[Martin Fick]

Awesome! Really neat stuff,

-Martin

On Tuesday, September 27, 2016 05:11:54 PM Saša Živkov
wrote:

--
The Qualcomm Innovation Center, Inc. is a member of Code
Aurora Forum, hosted by The Linux Foundation

[Edwin Kempin]

Thanks a lot for being a great host for the hackathon!

It was great to see you all!

--
--
To unsubscribe, email repo-discuss+unsubscribe@googlegroups.com

More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.

To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss+unsubscribe@googlegroups.com.