(bigtop-manager) branch main updated: BIGTOP-4169: The task log service did not send the completion signal (#26)

[wuzhiguo]

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

wuzhiguo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/bigtop-manager.git


The following commit(s) were added to refs/heads/main by this push:
     new 07d5830  BIGTOP-4169: The task log service did not send the completion 
signal (#26)
07d5830 is described below

commit 07d5830cb12e10153abf1997be4e5fc8f4828ed1
Author: Zhiguo Wu <wuzhi...@apache.org>
AuthorDate: Fri Jul 26 10:09:28 2024 +0800

    BIGTOP-4169: The task log service did not send the completion signal (#26)
---
 .../org/apache/bigtop/manager/agent/service/TaskLogServiceGrpcImpl.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/bigtop-manager-agent/src/main/java/org/apache/bigtop/manager/agent/service/TaskLogServiceGrpcImpl.java
 
b/bigtop-manager-agent/src/main/java/org/apache/bigtop/manager/agent/service/TaskLogServiceGrpcImpl.java
index 781cc3f..18d7269 100644
--- 
a/bigtop-manager-agent/src/main/java/org/apache/bigtop/manager/agent/service/TaskLogServiceGrpcImpl.java
+++ 
b/bigtop-manager-agent/src/main/java/org/apache/bigtop/manager/agent/service/TaskLogServiceGrpcImpl.java
@@ -58,6 +58,8 @@ public class TaskLogServiceGrpcImpl extends 
TaskLogServiceGrpc.TaskLogServiceImp
                 readNewLogs(file, responseObserver);
                 Thread.sleep(1000);
             }
+
+            responseObserver.onCompleted();
         } catch (Exception e) {
             String errMsg = "Error when reading task log: " + e.getMessage() + 
", please fix it";
             
responseObserver.onNext(TaskLogReply.newBuilder().setText(errMsg).build());