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

[GitBox <gi...@apache.org>]

fedexist edited a comment on issue #306: [LOG4J2-2678] Add LogEvent timestamp to Kafka ProducerRecord
URL: https://github.com/apache/logging-log4j2/pull/306#issuecomment-527152446
 
 
   I honestly wouldn't know whether there's any _actual_ use case where the producer timestamp (i.e. the current behaviour) is preferred to the event/record timestamp. Usually, if the record timestamp is not reliable, the timestamp is embedded within the message (e.g. a field in a JSON object) for it to be extracted by the processing framework being used.
   As I said, in my (relatively limited - 2 years, I admit) experience, the focus while using Kafka is the record timestamp and, moreover, the fact that, with log4j, we're in a true logging scenario makes it more plausible that someone using the KafkaAppender is actually interested to the LogEvent timestamp. 
   
   Still, making this an option is playing it safe, I guess.

----------------------------------------------------------------
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