ableegoldman commented on pull request #9066: URL: https://github.com/apache/kafka/pull/9066#issuecomment-663701977
@vvcephei I don't think there should be any issue there, since the checkpointed `OFFSET_UNKNOWN` is just ignored on read (or should be) and will never make it into the subscription info. That said, it does make me a bit nervous and I wouldn't count on this always being true, so I'd rather choose a different sentinel to be safe. Maybe we can put together a brief list of all sentinel values used in Streams (and the clients) to keep track and avoid issues like this going forward. I had to spend a long time digging around the client code to figure out whether an offset of -1 was ever possible in non-error cases (the answer seems to be "yes"). We'd have to actually keep it up to date of course, which might be...challenging ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
