Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

Log plugin#112

Closed
shargon wants to merge 1 commit intoneo-project:masterfrom
shargon:log-plugin
Closed

Log plugin#112
shargon wants to merge 1 commit intoneo-project:masterfrom
shargon:log-plugin

Conversation

@shargon
Copy link
Member

@shargon shargon commented Aug 2, 2019

@erikzhang
Copy link
Member

No. We should create a new plugin. "ApplicationLog` is for logging transaction execution results.

@shargon
Copy link
Member Author

shargon commented Aug 2, 2019

Maybe we should rename ApplicationLog, the expected behaviour is for logs 😆

@igormcoelho
Copy link
Contributor

Let's create a new plugin, SystemLog plugin, and use for this. ApplicationLog is less logging and more application data processing itself.

@shargon
Copy link
Member Author

shargon commented Aug 2, 2019

if we decide the name, i can make the changes

@shargon shargon closed this Aug 12, 2019
@shargon shargon deleted the log-plugin branch August 12, 2019 06:22
@shargon shargon mentioned this pull request Aug 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants