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
