tree bb578de1824421c02a7a9079b9392c552106054b
parent 90036bddc0a5b683565a5ad97501b0c99474cdd8
author Hugo Arès <hugo.ares@ericsson.com> 1504287036 -0400
committer Hugo Arès <hugo.ares@ericsson.com> 1504623439 -0400

Handle when uploader email matches more than one account

It's possible for an email to be used by more than one account. When the
uploader email matches more than one account, ChangeEventListener was
throwing an IllegalArgumentException preventing the operation from
completing successfully.

In most of the case, this is not harmful since ChangeEvent listeners
are called at the end of the operation but in the following scenario, it
makes gerrit data inconsistent.

If user push a new patchset to an existing change but bypass review, the
operation will be aborted by the IllegalArgumentException resulting in
the change being merged in the repo but its status still open.

Change-Id: Icc48fe4548fc46169bec3561ca490a199107af3b