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

chore(logging): Add more logging and trace tags for debugging #8490

Merged
merged 5 commits into from
Dec 7, 2022

Conversation

all-seeing-code
Copy link
Contributor

@all-seeing-code all-seeing-code commented Dec 6, 2022

Adds logging and sets query tag for improved tracing in Jaeger.

Damon Feldman and others added 4 commits December 6, 2022 16:59
(cherry picked from commit cd78f696fb4e8482a8a3127dd3ab7114c9e46417)
@coveralls
Copy link

coveralls commented Dec 6, 2022

Coverage Status

Coverage decreased (-0.002%) to 37.171% when pulling 1e5000d on anurag/Add-logging into 2b1441d on main.

edgraph/server.go Outdated Show resolved Hide resolved
Copy link
Contributor

@meghalims meghalims left a comment

Choose a reason for hiding this comment

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

We went over it and looks good.

@all-seeing-code all-seeing-code merged commit 1e06e81 into main Dec 7, 2022
@all-seeing-code all-seeing-code deleted the anurag/Add-logging branch December 7, 2022 19:27
all-seeing-code added a commit that referenced this pull request Dec 12, 2022
Adds logging and sets query `tag` for improved tracing in Jaeger.

Co-authored-by: Damon Feldman <[email protected]>
(cherry picked from commit 1e06e81)
@damonfeldman
Copy link

This enhancement ensures that the duration of all queries are logged if either the -v 3 logging level is set (level 3 is debug) or if "request logging" is enabled on the alpha. In addition, all query errors are logged if logging is set to -v 3. Previously, errors were not logged on the alpha (only reported to the client).

Also, for a particular error in zero logs reading "Error while writing: ..." the root cause of the error is now logged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants