Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/3278
Thanks for the detailed review @tillrohrmann, I'll follow-up and address
your comments.
Regarding removing the `setFlushOnCheckpoint`:
I think it was added at first to provide flexibility for users who know
what they are doing, and making sure that the producer will be able to work in
all environments (see comments in #2108).
However, recently I've also gathered opinions (from you and others) about
the settings over complicating at-least-once guarantees for the producer, and I
have the feeling we can remove it starting from the next release.
There is FLINK-5728 to enable flushing by default (currently the default is
no flushing). I'll incorporate your opinion on this to that JIRA, and decide
there if we only want to disable if by default or remove it completely.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
