[GitHub] [incubator-apisix] Akayeshmantha commented on issue #1358: Updating kafka logger to use the batch processor util

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

Akayeshmantha commented on issue #1358: Updating kafka logger to use the batch processor util
URL: https://github.com/apache/incubator-apisix/pull/1358#issuecomment-605695853
 
 
   Hi @sshniro  I thin Kafka logger does not needs to use the batch processor since it already has the capability to push to the topic in an asynchronous way. Lua resty Kafka already has the feature already inbuilt I already mentioned that in the documentation file of the logger
   
   The message will write to the buffer first. It will send to the Kafka server when the buffer exceeds the batch_num, or every flush_time flush the buffer in Async mode 
   
   cc :- @membphis wdyt ?

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