JunRuiLee commented on PR #516: URL: https://github.com/apache/paimon-rust/pull/516#issuecomment-4955964756
Thanks, both points make sense. This PR originally mirrored the `a50a36ff8` read kernel, but I agree the inactive-source behavior is an interop blocker against current Java master. I'll update this PR to adopt the current Java behavior for inactive ANN sources and add regression tests. I'll also replace the bucket merge Vec scan with a BinaryHeap as suggested. I'll keep larger current-master features like `GlobalIndexSearchMode` / FAST mode out of this PR to keep the fix scoped to the review comments. -- 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]
