Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug][LoggerServer] task log can not refresh in time #4617

Closed
597365581 opened this issue Jan 29, 2021 · 5 comments · Fixed by #4618
Closed

[Bug][LoggerServer] task log can not refresh in time #4617

597365581 opened this issue Jan 29, 2021 · 5 comments · Fixed by #4618
Labels
bug Something isn't working
Milestone

Comments

@597365581
Copy link
Contributor

for example ,if 1s produce log size can not teet the default conditions --Constants.DEFAULT_LOG_ROWS_NUM(64), and need to wait a long time。
image

@597365581 597365581 added the bug Something isn't working label Jan 29, 2021
597365581 added a commit to 597365581/incubator-dolphinscheduler that referenced this issue Jan 29, 2021
fix [Bug][LoggerServer] task log can not refresh in time apache#4617
@597365581
Copy link
Contributor Author

例如,如果 1s 内产生的日志量不能满足代码中默认的64条,并且后续又需要等很久才有新的日志产生。 就会导致 日志很久都无法被刷新。

@felix-thinkingdata
Copy link
Contributor

felix-thinkingdata commented Jan 29, 2021

for example ,if 1s produce log size can not teet the default conditions --Constants.DEFAULT_LOG_ROWS_NUM(64), and need to wait a long time。
image

image

This is where the content that has not been written to the log is uniformly disposed of after the task has completed.

Probably because of efficiency

@597365581
Copy link
Contributor Author

#4618

@597365581
Copy link
Contributor Author

for example ,if 1s produce log size can not teet the default conditions --Constants.DEFAULT_LOG_ROWS_NUM(64), and need to wait a long time。
image

image

This is where the content that has not been written to the log is uniformly disposed of after the task has completed.

Probably because of efficiency

so, if this task exec need long time, log can not write in time, inconvenient problem positioning。

@597365581
Copy link
Contributor Author

597365581 commented Feb 2, 2021

for example

#!/bin/bash
echo "hello world"
exec 10m
echo "hello world2"
if 1s produce log size can not meet the default conditions --Constants.DEFAULT_LOG_ROWS_NUM(64), and need to wait a long time。 because while ((line = inReader.readLine()) != null) is a block method。

davidzollo added a commit that referenced this issue Feb 24, 2021
[FIX-#4617][LoggerServer] task log can not refresh in time
@davidzollo davidzollo added this to the 1.3.6-release milestone Feb 24, 2021
davidzollo pushed a commit that referenced this issue Mar 30, 2021
#5074)

* [1.3.6-prepare][FIX-#4617][LoggerServer] task log can not refresh in time #4618

issue #4617
pr #4618

* code style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants