jordepic commented on issue #5649:
URL: 
https://github.com/apache/datafusion-comet/issues/5649#issuecomment-5543142750

   For #5647 I think both my original change and yours have some attached 
benchmarks. 5648 seems like an important one. I know @unikdahal also will 
eventually want merge into operations accelerated.
   
   Ah, another one I didn't cover - writes to MOR (merge on read) tables 
(positional deletes/deletion vectors). Iceberg rust didn't support these a 
while ago and it still doesn't. We could recreate them ourselves, but it seems 
dumb to jump the gun - I just have had a really hard time upstreaming to 
iceberg-rust so it doesn't leave me with a ton of confidence.


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