fresh-borzoni commented on PR #216: URL: https://github.com/apache/fluss-rust/pull/216#issuecomment-4219482836
@leekeiabstraction @linguoxuan Reviewed against the Java implementation in fluss. Found two bugs: response matching keyed by bucket_id alone (wrong results on partitioned tables) and sequential RPC sending. Fixed matching to use full TableBucket, also concurrent sending via FuturesUnordered. Added partitioned integration test and fixed the PR's test to compile against current main. https://github.com/fresh-borzoni/fluss-rust/tree/batch_skl -- 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]
