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

Consider adding a transaction ID to the logged messages #49

Open
awpratten opened this issue Jul 19, 2020 · 0 comments
Open

Consider adding a transaction ID to the logged messages #49

awpratten opened this issue Jul 19, 2020 · 0 comments
Labels
enhancement New feature or request Proposal Proposing a change or addition

Comments

@awpratten
Copy link
Member

From my commentary on the logger changes

The command/subsystem paradigm allows for a simulated parallel processing where you might have concurrent activities each initiated through separate events but interleaved in the logs. In other logging systems there is a transaction id that is included in the log so that you can see the flow and pull out only those events for the activity you care about. A contrived example To make my point would be intaking while manipulating the control panel. The transaction id could be generated from the initial action of the human operator. Something to consider adding if you feel it would help debugging.

I’d think that the transaction ID would be where the action is initiated. So if the operator initiates a climb then a transaction id would be acquired and used for all subsequent logging calls that follow directly from that action. Since the transaction id would need to be unique something in the logger should assign them. Or you could use utime() at the time of initiation or a random number although that won’t be sequential so maybe not as useful

@awpratten awpratten added enhancement New feature or request Proposal Proposing a change or addition labels Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Proposal Proposing a change or addition
Projects
None yet
Development

No branches or pull requests

1 participant