scwhittle commented on PR #37326: URL: https://github.com/apache/beam/pull/37326#issuecomment-3767890467
> We are wondering if the easy fix would be only call getNextReadChangeStreamEndTimestamp immediately before changeStreamQuery, like > https://github.com/apache/beam/pull/37338/files I would rather fix it like this PR, because otherwise we are still leaving the race possible (though less probable) if the thread is for some reason unscheduled for that long or if there is queuing inside the spanner client library. PTAL at the updated commit, I changed it to handle cases that we want to stop the restriction early such as out of range exception and the partition end record. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
