Satyr09 commented on PR #5789: URL: https://github.com/apache/datafusion-comet/pull/5789#issuecomment-5654089912
aah, sorry @andygrove just saw your new comment. Thanks for the super quick review! I've already updated the description and I'm currently fixing the test coverage issue you mentioned. Re: the newly discovered negative epoch timestamp bug - I've documented it here : https://github.com/apache/datafusion-comet/issues/5896 Would you prefer that that fix go in a separate PR? -- 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]
