[Improvement-17788] [TaskPlugin] Optimization of log processing for RemoteShellTask

[qiong-zhou]

Purpose of the pull request

Optimization of log processing for RemoteShellTask. Reading the logs by line to avoid large string. close #17788

Brief change log

Modified RemoteExecutor.java

Verify this pull request

This change added tests and can be verified as follows:

• Added testTrack() in RemoteExecutorTest to verify the change.

[qiong-zhou]

@SbloodyS WorkflowInstanceFailoverTestCase.testGlobalFailover_runningWorkflow_fromAnotherMaster failed.
This unit test failure appears to be unrelated to my changes. Please help me out.

[SbloodyS]

@SbloodyS WorkflowInstanceFailoverTestCase.testGlobalFailover_runningWorkflow_fromAnotherMaster failed. This unit test failure appears to be unrelated to my changes. Please help me out.

I've already rerun all tests.

[qiong-zhou]

@SbloodyS WorkflowInstanceFailoverTestCase.testGlobalFailover_runningWorkflow_fromAnotherMaster failed. This unit test failure appears to be unrelated to my changes. Please help me out.

I've already rerun all tests.

@SbloodyS Unit tests failed again, any idea what might be causing this?

[ruanwenjun]

@qiong-zhou CI issue fixed by #17807

[qiong-zhou]

@ruanwenjun ok thanks.
@SbloodyS I've merged #17807 Please rerun the checks.
I'm wandering if I can rerun checks by myself?

[ruanwenjun]

@ruanwenjun ok thanks. @SbloodyS I've merged #17807 Please rerun the checks. I'm wandering if I can rerun checks by myself?

Right now, the CI need committer approve, this is primarily to reduce unnecessary CI resource consumption. If we can move the CI action to developer's repo, then we can remove this.

[qiong-zhou]

@SbloodyS Please review the code.

[SbloodyS]

LGTM

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud