Fix Kafka flush() contract different in vanilla client v/s li-apache-kafka-clients

linkedin / li-apache-kafka-clients

li-apache-kafka-clients is a wrapper library for the Apache Kafka vanilla clients. It provides additional features such as large message support and auditing to the Java producer and consumer in the open source Apache Kafka.

BSD 2-Clause "Simplified" License

131 stars 52 forks source link

Fix Kafka flush() contract different in vanilla client v/s li-apache-kafka-clients #192

Closed fluffywei closed 3 years ago

fluffywei commented 3 years ago

This request aims to fixing li-apache-kafka-clients producer flush() contract different in vanilla client.

Removed verifyOpen() from flush() and flush(long timeout, TimeUnit timeUnit).
Peeled InvocationTargetException and re-throw vanilla client exception.
Added flush() integration test.

fluffywei commented 3 years ago

@kehuum