Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Once a change has a +2 and has been verified by a pre-merge verification run, it can be submitted to gerrit. Once it is in is gerrit, a committer needs to push the commit to the Apache git repository, by running:

 

Code Block
bin/push_to_asf.py

push_to_asf.py checks the latest commits in gerrit and checks if they are in the Apache git repo. If some are not in the Apache git repo yet, it will ask you if you want to update the Apache git repo with the missing commits found in the Gerrit repo. It does not check what your local state is at all. It only compares remote Gerrit with the remote Apache repo. Keep in mind that if you are a committer, it will allow you to commit any change authored by anyone that has passed GVO.

...

As described above, Gerrit generates one review request per new commit. It's much easier to review everything related to a patch together in one single commit, so please make sure to squash all your related commits together before submitting. If you don't, your review requests will probably get forcibly removed

 

Other Useful Tools and Information

Searching for all Impala reviews

You might want to bookmark this link, which shows all open reviews for Impala.

git-review (Python Gerrit helper module)

git-review is a Python module that can simplify publishing gerrit reviews. To install it use:

Code Block
sudo pip install git-review

By default, git-review will look for a remote named 'gerrit'. The usage is:

Code Block
# Submit a code review (or update an existing review). This will also do a git rebase -i asf-gerrit/master.
git review -r asf-gerrit

git-review can be used to apply changes from a review locally and compare patch sets. For a full set of options see: https://pypi.python.org/pypi/git-review

Gerrit documentation

The documentation for Gerrit is here:

http://gerrit.cloudera.org/Documentation/index.html