You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@logging.apache.org by GitBox <gi...@apache.org> on 2019/09/02 02:25:32 UTC

[GitHub] [logging-log4j2] rgoers edited a comment on issue #306: [LOG4J2-2678] Add LogEvent timestamp to Kafka ProducerRecord

rgoers edited a comment on issue #306: [LOG4J2-2678] Add LogEvent timestamp to Kafka ProducerRecord
URL: https://github.com/apache/logging-log4j2/pull/306#issuecomment-526979931
 
 
   @garydgregory How could making it optional possibly break existing apps? Making it optional would be done to preserve the existing behavior and not break existing apps and is the only reason one would want to make it optional.  
   Personally, I think it should be optional but the default should be to include it. This would give most people their desired behavior by default but would allow users to preserve the existing behavior if they need to.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services