Skip to content

Improve logging routine in runtime engine #641

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

Open
petekanev opened this issue Nov 28, 2016 · 0 comments
Open

Improve logging routine in runtime engine #641

petekanev opened this issue Nov 28, 2016 · 0 comments
Labels

Comments

@petekanev
Copy link
Contributor

Debug log messages need to be improved by providing additional information, such as TID, Category, and provide a better verbosity overall. This will allow us to spot bugs more easily when reported by users, as well as to pinpoint potential bottlenecks in the engine.

The improved logging should provide more granular control of the logs, or just categorization of the different execution paths in the runtime. It can be modeled after the Tracing routine in .NET -> https://msdn.microsoft.com/en-us/library/sk36c28t(v=vs.110).aspx

Trace (Log) statements throughout the runtime code (Instrumentation of the code) are categorized with a relevant preset category. If a listener is attached for that category, messages will be output as per the Listener's implementation. - Default LogCat implementation should be available for all categories.

Can be enabled by setting the "android": { "diagnosticTracing": true } in the project's package.json

Micro benchmarking can also be enabled to record time spent in execution of most key methods (that are measurable, can be measured with little to no error added by resolution timing - time spent to record start and end time). The benchmarking routine should output logs in a parsable manner for easier visualization.

Can be enabled by setting the "android": { "diagnosticBenchmarking": true } in the project's package.json

Enabling tracing could and probably WILL impose a performance penalty

@petekanev petekanev added this to the 2.5.0-RC milestone Nov 28, 2016
@petekanev petekanev self-assigned this Nov 28, 2016
@petekanev petekanev removed this from the 2.5.0-RC milestone Jan 4, 2017
@petekanev petekanev removed their assignment May 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants