This is an automated email from the ASF dual-hosted git repository.

guanhuali pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new be82be78cc [ZEPPELIN-5893] NPE in ParagraphResult when getting 
progress from paragraphJson (#4579)
be82be78cc is described below

commit be82be78ccb78f56cca7b12fe88583e796b1c637
Author: zzm <xiaomin0...@sina.com>
AuthorDate: Thu May 4 11:13:46 2023 +0800

    [ZEPPELIN-5893] NPE in ParagraphResult when getting progress from 
paragraphJson (#4579)
    
    * Update ParagraphResult.java
    
    paragraphJson.getInt("progress")  Add Judgment
    
    * Update ParagraphResult.java
    
    format
    
    * Update ParagraphResult.java
    
    Optimize code indentation
---
 .../src/main/java/org/apache/zeppelin/client/ParagraphResult.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/zeppelin-client/src/main/java/org/apache/zeppelin/client/ParagraphResult.java 
b/zeppelin-client/src/main/java/org/apache/zeppelin/client/ParagraphResult.java
index a28c604a9b..80872defc6 100644
--- 
a/zeppelin-client/src/main/java/org/apache/zeppelin/client/ParagraphResult.java
+++ 
b/zeppelin-client/src/main/java/org/apache/zeppelin/client/ParagraphResult.java
@@ -45,7 +45,9 @@ public class ParagraphResult {
   public ParagraphResult(JSONObject paragraphJson) {
     this.paragraphId = paragraphJson.getString("id");
     this.status = Status.valueOf(paragraphJson.getString("status"));
-    this.progress = paragraphJson.getInt("progress");
+    if (paragraphJson.has("progress")) {
+      this.progress = paragraphJson.getInt("progress");
+    }
     this.results = new ArrayList<>();
     if (paragraphJson.has("results")) {
       JSONObject resultJson = paragraphJson.getJSONObject("results");

Reply via email to