[jira] [Created] (KAFKA-6325) Producer.flush() doesn't throw exception on timeout

["Erik Scheuter (JIRA)" <ji...@apache.org>]

Erik Scheuter created KAFKA-6325:
------------------------------------

             Summary: Producer.flush() doesn't throw exception on timeout
                 Key: KAFKA-6325
                 URL: https://issues.apache.org/jira/browse/KAFKA-6325
             Project: Kafka
          Issue Type: Bug
          Components: producer 
            Reporter: Erik Scheuter
         Attachments: FlushTest.java

Reading the javadoc of the flush() method we assumed an exception would've been thrown when an error occurs. This would make the code more understandable as we don't have to return a list of futures if we want to send multiple records to kafka and eventually call future.get().

When send() is called, the metadata is retrieved and send is blocked on this process. When this process fails (no brokers) an FutureFailure is returned. 

When you just flush; no exceptions will be thrown (in contrast to future.get()). Ofcourse you can implement callbacks in the send method.

I think there are two solutions:
* Change flush() (& doSend()) and throw exceptions
* Change the javadoc and describe the scenario you can lose events because no exceptions are thrown and the events are not sent.

I added an unittest to show the behaviour. Kafka doesn't have to be available for this.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)