comphead commented on PR #395:
URL: https://github.com/apache/datafusion-comet/pull/395#issuecomment-2098828743

   > > should we get this covered by tests
   > 
   > This is for iceberg/comet integration. I don't think there is an easy way 
to test this now. I have tested this on my local, though.
   
   Maybe we can return uuid value and assert it somehow, although its 
non-determenistic? I'm thinking if we can be protected from regression if 
anyone else changes this code later? 


-- 
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: github-unsubscr...@datafusion.apache.org

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


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

Reply via email to