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

Add descriptive logs to starter kits #42

Open
3 tasks
yash-puligundla opened this issue Aug 4, 2022 · 1 comment
Open
3 tasks

Add descriptive logs to starter kits #42

yash-puligundla opened this issue Aug 4, 2022 · 1 comment
Assignees

Comments

@yash-puligundla
Copy link
Member

  • Any “throw new exception” (ex: line 95 in wes) should have an "error" log that prints the exception stack trace/ message before throwing the generic exception (like “Could not load WES run log”)
  • Add "debug" level logs to commonly/ most used pieces of code. I would start with controllers. The idea is to be able to figure out the point/ region of code where an error might happen & for which object_id or input parameters, so that it's easier to reproduce this error while debugging.
  • This is totally optional and I would leave it to the very end. I have implemented struct logs that direct all the logs into a file. (Feel free to implement/ignore this as it is not a priority feature for the logs issue you are working on) - https://github.com/ga4gh/ga4gh-testbed-api/blob/develop/src/main/java/org/ga4gh/testbed/api/utils/logging/TestbedLoggingUtil.java
@muhd360
Copy link

muhd360 commented Nov 14, 2024

@yash-puligundla can i take up this issue??

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

No branches or pull requests

3 participants