u70b3 commented on issue #9035: URL: https://github.com/apache/paimon/issues/9035#issuecomment-5181994582
take — I'd like to fix this. For context: I first ran into this while implementing `scan.watermark` batch time travel in paimon-rust (apache/paimon-rust#677). Cross-validating the Rust behavior against the Java reference implementation surfaced these defects in the Java SDK itself; the Rust side avoids them structurally (bounded in-window fallback, binary search over the actual snapshot id list, `None`/`i64::MIN` treated uniformly as missing). I'll submit a PR fixing both `laterOrEqualWatermark` and `earlierOrEqualWatermark` (including the inverted early-return at :391) along the lines described in the issue, with tests covering the interleaved-null, all-null and below-minimum cases. -- 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]
