You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by GitBox <gi...@apache.org> on 2022/03/21 18:11:19 UTC

[GitHub] [flink-kubernetes-operator] gyfora opened a new pull request #90: [FLINK-26454] Clean up logging and use mdc in template

gyfora opened a new pull request #90:
URL: https://github.com/apache/flink-kubernetes-operator/pull/90


   This PR improves logging by removing adhoc name/namespace information from the logged strings and replaces it with context information from the MDC to apply to all logged lines uniformly.
   
   Example output after the change:
   ```
   flink-operator 2022-03-21 18:08:52,205 o.a.f.k.o.o.JobObserver        [INFO ][default/basic-example] Job status successfully updated
   flink-operator 2022-03-21 18:08:52,205 o.a.f.k.o.o.JobObserver        [DEBUG][default/basic-example] Savepoint not in progress
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #90: [FLINK-26454] Clean up logging and use mdc in template

Posted by GitBox <gi...@apache.org>.
morhidi commented on pull request #90:
URL: https://github.com/apache/flink-kubernetes-operator/pull/90#issuecomment-1074748405






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [flink-kubernetes-operator] gyfora merged pull request #90: [FLINK-26454] Clean up logging and use mdc in template

Posted by GitBox <gi...@apache.org>.
gyfora merged pull request #90:
URL: https://github.com/apache/flink-kubernetes-operator/pull/90


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [flink-kubernetes-operator] gyfora commented on pull request #90: [FLINK-26454] Clean up logging and use mdc in template

Posted by GitBox <gi...@apache.org>.
gyfora commented on pull request #90:
URL: https://github.com/apache/flink-kubernetes-operator/pull/90#issuecomment-1074247021


   cc @morhidi 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [flink-kubernetes-operator] morhidi removed a comment on pull request #90: [FLINK-26454] Clean up logging and use mdc in template

Posted by GitBox <gi...@apache.org>.
morhidi removed a comment on pull request #90:
URL: https://github.com/apache/flink-kubernetes-operator/pull/90#issuecomment-1074748405


   +1, LGTM, we can also consider setting the root log level to INFO in the operator


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org