PostReview: update and insert comments/approvals in a single step

Conflicts:
	gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java

Change-Id: Ieab4ac0628e397495bfd8a278c879f79817ba443
Update 2.8.2 release notes

Change-Id: Ia580893c2c1700af9b6eddc6ee9a7916e68aa551
Bump GERRIT_VERSION to 2.8.2

Change-Id: I24e6deb8755fda9d90181db6a35397236f6c789f
Prevent duplicate commits in same project when uploading to refs/changes/n

Under certain circumstances, when pushing to 'refs/changes/n', the same
commit can be pushed onto multiple changes even if they are on the same
branch.

As a result of this, if the commit sha1 is used as change identifier
in the 'review' ssh command, it will fail with the error:

  'fatal: <SHA1> matches multiple patch sets.'

This commit performs the same check as when pushing to 'refs/for/branch'
to prevent this, and will show "commit already exists (in the project)"
error message to reject uploading a commit to an existing change via
`refs/changes/n` if the commit was already successfully pushed to a change
in project scope.

Bug: issue 2374
Change-Id: I1c000b2cc3e155617cdc9c295a46ce2107ec47ca
Bump version to 2.8.2 in plugin API and archetypes

Change-Id: Iaa71fe340597703b2effd60369aecd09335f4d3e
Update 2.8.2 release notes

Change-Id: I120587908d05e61a87c6058a481c6085f9586c3e
Buck: Expand instructions related to setting up the path

Adding the ~/bin folder to the path in the current shell session
is not enough.  It also needs to be added globally.

Update the documentation to mention this.

Change-Id: Ic5d4cf1a02b3e52eccb6e34ba3513a69785df1f9
Fix formatting problem in buck documentation

The bullet list for API artifact repository types was not being
rendered correctly due to a missing blank line.

Also, add backticks around the version numbers in the list.

Change-Id: I58d8845acde464d223e7838160be8d222e059f2b
Fix typos in 2.8.2 release notes

Bug: Issue 2547
Change-Id: Ic767e0cfdef01d335eb7d4e58bc79f80348931f0
Fix cherry pick of multiple changes

Second and additional changes in the same submit attempt must be built
on top of the previously cherry picked change, not the branch tip.

Change-Id: I9183f68b5780af1f28cb357217663e4bdb2d4e76
Merge changes I58d8845a,Ic5d4cf1a into stable-2.8

* changes:
  Fix formatting problem in buck documentation
  Buck: Expand instructions related to setting up the path
Fix Merge Always and Merge If Necessary strategies

Second and additional changes in the same submit attempt must be
built on the previously constructed merge, not the branch tip.

Include tests for these strategies and the same bug in Cherry Pick
that was fixed by cd60499cd65812d926806e49d15a6852c7798871.

Change-Id: I62524b39ee4104ab53b36c75b64c02b20b4d5e94
Bump GERRIT_VERSION to 2.8.3

Change-Id: I7dd789d55d222ea66a3f52ca140412f169e7f976
Bump version to 2.8.3 in plugin API and archetypes

Change-Id: I9c514489f31d223193139516c1d7a8ce059525e9
Update cookbook plugin revision

Sets the correct API version in the pom.xml file.

Change-Id: I662d4dd2f6c2a18c64cc01cdb6193e24dc0fbad9
Add clarification about commitWithin setting and reindex

If the commitWithin setting is low, it can cause the reindex to
run slowly.  Add a note in the documentation about this.

Change-Id: If4c394443bde89be4bfc1505d33322089e671493
Release notes for Gerrit 2.8.3

Change-Id: Ic0bd16b0a0f174aac7039ede83e557750e506217
Reformat index documentation

Move the paragraphs about running the reindex program and the index
names up to the top of the index section.  It makes more sense to
have them there, than in the middle of the setting descriptions.

Change-Id: I875a66173f752554803ca95bc166fba0dfbcd315
Add documentation of the index.threads setting

Change-Id: I4e7b737791968123d78c19ef558327609ef11b30
Escape dollar sign in plugin manifest entries

Escape dollar sign in plugin manifest entries, to prevent that
it is interpreted by the shell and thus get removed.

Change-Id: Iefd8abdcd207cb985f8156c7e41c094c1aa12b27