opendev/gerrit
Code Issues Proposed changes

Rebase: Don't swallow caught exception

Browse Source 
Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Ie485ec39ae7acc2863823fefa39494475c302079
This commit is contained in:
Edwin Kempin
2019-08-22 10:10:15 +02:00
committed by David Pursehouse
parent aefd1d5c27
commit e4d1b55a1a
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-extension-api/src/main/java/com/google/gerrit/extensions/restapi/UnprocessableEntityException.java
View File

@@ -21,4 +21,8 @@ public class UnprocessableEntityException extends RestApiException {
public UnprocessableEntityException(String msg) {
super(msg);
}
public UnprocessableEntityException(String msg, Throwable cause) {
super(msg, cause);
}
}

2
gerrit-server/src/main/java/com/google/gerrit/server/change/Rebase.java
View File

@@ -156,7 +156,7 @@ public class Rebase extends RetryingRestModifyView<RevisionResource, RebaseInput
}
} catch (NoSuchChangeException e) {
throw new UnprocessableEntityException(
String.format("Base change not found: %s", input.base));
String.format("Base change not found: %s", input.base), e);
}
PatchSet.Id baseId = base.patchSet().getId();