szlta commented on issue #4292:
URL: https://github.com/apache/iceberg/issues/4292#issuecomment-1090040631

   Hi @aokolnychyi sorry for the longer pause here, I was on paternity leave ;)
   Before I left, I was looking into how proposal 2 in 
https://github.com/apache/iceberg/pull/3411#discussion_r823692312 could be 
implemented in Iceberg. It seemed to be quite tricky, so first I'd jump onto 
the smaller fixes of the above proposed list.
   I'd be happy to take this up and work on a PR that should fix these two 
points:
   
   - Use Partitioning$partitionType in all metadata tables.
   - Consider adding partition_spec_id to metadata tables where this is missing 
(I think it's only the partitions meta table).
   
   These two relate strongly, so I think they are worth handling in the same 
change request. I'll start working on this PR today.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to