Skip to content

Commit

Permalink
fix: Check for null result in error handling (deephaven#5808)
Browse files Browse the repository at this point in the history
  • Loading branch information
jmao-denver committed Jul 18, 2024
1 parent 7ee9aed commit 6ca0c89
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ protected void propagateError(
}

protected boolean systemicResult() {
return SystemicObjectTracker.isSystemic(result);
return result == null ? false : SystemicObjectTracker.isSystemic(result);
}

@Override
Expand All @@ -172,6 +172,9 @@ protected void destroy() {

protected void propagateErrorDownstream(
final boolean fromProcess, @NotNull final Throwable error, @Nullable final TableListener.Entry entry) {
if (result == null) {
return;
}
if (fromProcess && result.satisfied(getUpdateGraph().clock().currentStep())) {
// If the result is already satisfied (because it managed to send its notification, or was otherwise
// satisfied) we should not send our error notification on this cycle.
Expand Down

0 comments on commit 6ca0c89

Please sign in to comment.