ludlows opened a new pull request, #14072:
URL: https://github.com/apache/iceberg/pull/14072

   with the sql :
   
   
   ```sql
   -- ddl
   create table db.tst (id int, partition int) USING iceberg PARTITIONED BY 
(partition) 
   TBLPROPERTIES ( 'format-version' = '2', 'write.delete.mode' = 
'merge-on-read');
   --data initialized
   insert into db.tst values (1, 1), (2, 1), (3, 2), (2, 2) ;
   --position delete
   update db.tst set partition=2 where id=1 ;
   --check metadata table but broken before this
   select * from db.tst.position_deletes ;
   ```
   the metadata table `db.tst.position_deletes`  is broken under the function 
`checkAndAddPartitionName`,
   since the schema is:
   ```
   schema: table {
     2147483546: file_path: required string (Path of a file in which a deleted 
row is stored)
     2147483545: pos: required long (Ordinal position of a deleted row in the 
data file)
     2147483544: row: optional struct<1: id: optional int, 2: partition: 
optional int> (Deleted row values)
     2147483642: partition: required struct<3: partition: optional int> 
(Partition that position delete row belongs to)
     2147483643: spec_id: required int (Spec ID used to track the file 
containing a row)
     2147483646: delete_file_path: required string (Path of the file in which a 
row is stored)
   }
   ```
   we can use fieldID to find the true column ID and fix this problem.
   
   related issue: 
   https://github.com/apache/iceberg/issues/14056 
   


-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to