fd4s / fs2-kafka

Functional Kafka Streams for Scala

Home Page:https://fd4s.github.io/fs2-kafka

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

CE 3.5.0 async cancelation semantics

Jasper-M opened this issue · comments

I think fs2-kafka will require some changes to accommodate the new async cancelation semantics in the upcoming CE 3.5.0.

For instance this timeout will probably be rendered useless. Also I suspect that currently this timeout only cancels the waiting on the callback but will leak the actual hanging commit, which is what those new semantics are guarding against.

Thanks for flagging this! I'm on it.

Also I suspect that currently this timeout only cancels the waiting on the callback but will leak the actual hanging commit, which is what those new semantics are guarding against.

Yes, you're right - unfortunately I don't think there's anything we can do about that unless we revert to using the commitSync method which has an optional timeout parameter.

I guess this is fixed, but github didn't close it for some reason?