You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uniffle.apache.org by ro...@apache.org on 2023/02/25 08:51:05 UTC
[incubator-uniffle] branch master updated: [#659] fix(server): fix NPE of ShuffleDataFlushEvent's underStorage in some cases (#660)
This is an automated email from the ASF dual-hosted git repository.
roryqi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git
The following commit(s) were added to refs/heads/master by this push:
new c70d1543 [#659] fix(server): fix NPE of ShuffleDataFlushEvent's underStorage in some cases (#660)
c70d1543 is described below
commit c70d1543e7c9ca774e986026890763ce949a10e5
Author: Junfan Zhang <zu...@apache.org>
AuthorDate: Sat Feb 25 16:51:00 2023 +0800
[#659] fix(server): fix NPE of ShuffleDataFlushEvent's underStorage in some cases (#660)
### What changes were proposed in this pull request?
fix NPE of ShuffleDataFlushEvent's underStorage in some cases
### Why are the changes needed?
Fix: #659
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing UTs
---
.../src/main/java/org/apache/uniffle/server/ShuffleDataFlushEvent.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/src/main/java/org/apache/uniffle/server/ShuffleDataFlushEvent.java b/server/src/main/java/org/apache/uniffle/server/ShuffleDataFlushEvent.java
index 6829ab0a..8dab3ac6 100644
--- a/server/src/main/java/org/apache/uniffle/server/ShuffleDataFlushEvent.java
+++ b/server/src/main/java/org/apache/uniffle/server/ShuffleDataFlushEvent.java
@@ -160,7 +160,7 @@ public class ShuffleDataFlushEvent {
+ ", startPartition=" + startPartition
+ ", endPartition=" + endPartition
+ ", retryTimes=" + retryTimes
- + ", underStorage=" + underStorage.getClass().getSimpleName()
+ + ", underStorage=" + (underStorage == null ? null : underStorage.getClass().getSimpleName())
+ ", isPended=" + isPended;
}
}