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

   ### Purpose
   
   Linked issue: close #185 
   
   Fix issue with TableLookup parsing being off by 2 columns
   
   ### Brief change log
   
   Fix TableLookup so that SchemaId field bytes are not passed to 
CompactedRow::from_bytes(), these fields can be skipped as current rust client 
implementation does not check schema and already passes row_type
   
   ### Tests
   
   Verified using `example_kv_table.rs`. Integration test to follow
   
   ```
   === Upserting ===
   Upserted: GenericRow { values: [Int32(1), String("Verso"), Int64(32)] }
   Upserted: GenericRow { values: [Int32(2), String("Noco"), Int64(25)] }
   Upserted: GenericRow { values: [Int32(3), String("Esquie"), Int64(35)] }
   
   === Looking up ===
   Found id=1: name=Verso, age=32
   Found id=2: name=Noco, age=25
   Found id=3: name=Esquie, age=35
   
   === Updating ===
   Updated: GenericRow { values: [Int32(1), String("Verso"), Int64(33)] }
   Verified update: name=Verso, age=33
   
   === Deleting ===
   Deleted: GenericRow { values: [Int32(2), String(""), Int64(0)] }
   Verified deletion
   ```
   


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