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.

Sign up for GitHub

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

Jump to bottom

[INLONG-4824][Log] Print line numbers in log info for all modules #4825

Merged
merged 2 commits into from
Jul 1, 2022
Merged

[INLONG-4824][Log] Print line numbers in log info for all modules #4825

merged 2 commits into from
Jul 1, 2022

Conversation

Greedyu
Copy link
Contributor

@Greedyu Greedyu commented Jun 30, 2022

Add line numbers to Agent module log output

@healchow healchow changed the title [INLONG-4824][Agent] Add line numbers to Agent module log output [INLONG-4824][Agent] Print line numbers in log info Jun 30, 2022
@dockerzhang
Copy link
Contributor

@Greedyu please check the log4j configurations for the DataProxy and Manager at the same time, thanks.

@Greedyu
Copy link
Contributor Author

Greedyu commented Jul 1, 2022

configurations

resolve

@dockerzhang dockerzhang changed the title [INLONG-4824][Agent] Print line numbers in log info [INLONG-4824][Log] Print line numbers in log info Jul 1, 2022
@dockerzhang dockerzhang changed the title [INLONG-4824][Log] Print line numbers in log info [INLONG-4824][Log] Print line numbers in log info for all modules Jul 1, 2022
pocozh
pocozh approved these changes Jul 1, 2022
View reviewed changes
Copy link
Contributor

@pocozh pocozh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@dockerzhang dockerzhang merged commit bc9a0d9 into apache:master Jul 1, 2022
vernedeng pushed a commit to vernedeng/incubator-inlong that referenced this pull request Jul 4, 2022
@Greedyu @vernedeng
[INLONG-4824][Log] Print line numbers in log info for all modules (ap…
3a1015f 
…ache#4825)
bruceneenhl pushed a commit to bruceneenhl/inlong that referenced this pull request Aug 12, 2022
@Greedyu @bruceneenhl
[INLONG-4824][Log] Print line numbers in log info for all modules (ap…
3661fa0 
…ache#4825)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dockerzhang dockerzhang dockerzhang approved these changes

@healchow healchow healchow approved these changes

@pocozh pocozh pocozh approved these changes

Assignees

@Greedyu Greedyu

Labels
component/agent component/audit component/dataproxy component/manager component/tubemq
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Improve][Agent] Log output has no line numbers
4 participants
@Greedyu @dockerzhang @healchow @pocozh