-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[pulsar function] add instanceId and fqn into log message properties #11399
Conversation
Thanks for your contribution. For this PR, do we need to update docs? (The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks) |
There's no doc about the existing property for log topic. It's good to mention the available propperties for a log message, but I don't know where to add this. |
/pulsarbot run-failure-checks |
@nlu90 thanks. You can ping me if you need a doc review. |
/pulsarbot run-failure-checks |
fb1bbda
to
90058a0
Compare
/pulsarbot run-failure-checks |
@nlu90 you might need to rebase to the latest master to include CI fixes. |
90058a0
to
4ba76df
Compare
/pulsarbot run-failure-checks |
1 similar comment
/pulsarbot run-failure-checks |
### Motivation Enrich the published log message with `instanceId` and `fqn` set in properties. ### Modifications add `instanceId` and `fqn` into log message properties (cherry picked from commit b6d02e4)
### Motivation Enrich the published log message with `instanceId` and `fqn` set in properties. ### Modifications add `instanceId` and `fqn` into log message properties
Motivation
Enrich the published log message with
instanceId
andfqn
set in properties.Modifications
add
instanceId
andfqn
into log message propertiesVerifying this change