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

IWF-105: Add event logger #480

Merged
merged 11 commits into from
Nov 18, 2024
Merged

IWF-105: Add event logger #480

merged 11 commits into from
Nov 18, 2024

Conversation

lwolczynski
Copy link
Contributor

Description

Checklist

  • Code compiles correctly
  • Tests for the changes have been added
  • All tests passing
  • This PR change is backwards-compatible
  • This PR CONTAINS a (planned) breaking change (it is not backwards compatible)

Related Issue

Closes #issue_number

@lwolczynski lwolczynski changed the title Jira/lwolczynski/iwf 105 IWF-105: Add event logger Nov 14, 2024
@lwolczynski lwolczynski force-pushed the jira/lwolczynski/IWF-105 branch from 6f2395e to 8bec18e Compare November 14, 2024 21:53
@lwolczynski lwolczynski force-pushed the jira/lwolczynski/IWF-105 branch from 7891697 to 2afa674 Compare November 18, 2024 19:24
@lwolczynski lwolczynski marked this pull request as ready for review November 18, 2024 19:51
import "github.com/indeedeng/iwf/gen/iwfidl"

// The implementation must be lightweight, reliable and fast (less than 1s)
type LogEventFunc func(event iwfidl.IwfEvent)
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry for changing my mind on naming -- since this is event, maybe calling it "HandleEventFunc"?

because it may not be logging -- we may be sending to Kafka and consuming for DB etc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@longquanzheng
Copy link
Contributor

Lgtm otherwise

@lwolczynski lwolczynski force-pushed the jira/lwolczynski/IWF-105 branch from 78e3985 to 72d1587 Compare November 18, 2024 21:26
@@ -0,0 +1,16 @@
package handleevent
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe package event?

An the other places it's event.Handler(...)

@lwolczynski lwolczynski merged commit cb0d67c into main Nov 18, 2024
10 checks passed
@lwolczynski lwolczynski deleted the jira/lwolczynski/IWF-105 branch November 18, 2024 22:59
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

Successfully merging this pull request may close these issues.

3 participants