My main concern is different semantics for different methods: the old methods 
talking `long/long` have point-in-time/point-in-time semantics, while the new 
API has point-in-time/duration semantics. I would like to understand the impact 
of this change.

Especially, for `WindowedStore` we will now have both method (`long/long` is 
only deprecated on `ReadOnlyWindowedStore` but newly "added" to 
`WindowedStore`). It seems reasonable to me, to provide the same semantics for 
both -- overall, `long/long` should be a more performance runtime version of 
`Instant/Xxxx` and thus, offering different semantics using `Instant/Duration` 
(instead of `Instant/Instant`) might confuse users.

@nicktrav It seem to me, that we should discuss this carefully and maybe not 
rush into `2.1` release... Thoughts?

[ Full content available at: https://github.com/apache/kafka/pull/5682 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to