AndreaBozzo opened a new pull request, #159:
URL: https://github.com/apache/fluss-rust/pull/159

   ### Purpose
   
   Linked issue: close #114
   
   Implements lookup functionality to verify put kv operations. This follows 
the simple lookup approach as suggested in the issue comments (without complex 
batching/queuing).
   
   ### Brief change log
   
   - Add `Lookup` API key (1017) to `api_key.rs` - matching [Java Fluss 
ApiKeys.java](https://github.com/alibaba/fluss/blob/main/fluss-common/src/main/java/com/alibaba/fluss/rpc/protocol/ApiKeys.java)
   - Add lookup proto messages (`LookupRequest`, `LookupResponse`, 
`PbLookupReqForBucket`, `PbLookupRespForBucket`, `PbValue`) to `fluss_api.proto`
   - Create `lookup.rs` message module with `LookupRequest` struct implementing 
`RequestBody` trait
   - Add async `lookup(bucket_id, key)` method to `FlussTable` that returns 
`Option<Vec<u8>>`
   
   Reference Java implementation:
   - 
[LookupClient.java](https://github.com/alibaba/fluss/blob/main/fluss-client/src/main/java/com/alibaba/fluss/client/lookup/LookupClient.java)
   - 
[PrimaryKeyLookuper.java](https://github.com/alibaba/fluss/blob/main/fluss-client/src/main/java/com/alibaba/fluss/client/lookup/PrimaryKeyLookuper.java)
   
   ### Tests
   
   <!-- TODO: Add integration tests for lookup functionality -->
   
   ### API and Format
   
   Adds new public API:
   - `FlussTable::lookup(bucket_id: i32, key: Vec<u8>) -> 
Result<Option<Vec<u8>>>`
   
   ### Documentation
   
   No documentation changes required at this time.


-- 
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]

Reply via email to