CTTY commented on issue #1572:
URL: https://github.com/apache/iceberg-rust/issues/1572#issuecomment-3353894245

   Hi @mnpw , sorry that I didn't keep this issue updated, I actually have a 
draft already for this that is not published yet
   
   I'm planning to first refactor the existing writers and introduce the 
partition key, and we will also need another PartitioningWriter trait running 
on top of IcebergWriter. So the fanout writer will likely to have 
HashMap<partition_data, dyn IcebergWriter>. 
   
   There are some nuances about the generics types across different layers of 
writers that I haven't fully thought through, but will continue to polish it 
once the refactoring work is done :) 


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