andygrove commented on PR #1623:
URL:
https://github.com/apache/datafusion-comet/pull/1623#issuecomment-2790485519
Thanks for the reviews @mbutrovich and @parthchandra
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
andygrove merged PR #1623:
URL: https://github.com/apache/datafusion-comet/pull/1623
--
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...@
andygrove commented on PR #1623:
URL:
https://github.com/apache/datafusion-comet/pull/1623#issuecomment-2787831345
@mbutrovich @parthchandra this is ready for review now
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
codecov-commenter commented on PR #1623:
URL:
https://github.com/apache/datafusion-comet/pull/1623#issuecomment-2786822989
##
[Codecov](https://app.codecov.io/gh/apache/datafusion-comet/pull/1623?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_ca
mbutrovich commented on PR #1623:
URL:
https://github.com/apache/datafusion-comet/pull/1623#issuecomment-2786751017
> @mbutrovich How can I improve this test to expose current known issues
around INT96 handling?
The big thing is generating a timestamp that's larger than a 64-bit
nan
andygrove commented on PR #1623:
URL:
https://github.com/apache/datafusion-comet/pull/1623#issuecomment-2786664083
@mbutrovich How can I improve this test to expose current known issues
around INT96 handling?
--
This is an automated message from the Apache Git Service.
To respond to the