dhegberg opened a new pull request, #103: URL: https://github.com/apache/arrow-testing/pull/103
Adding data to support testing for Nulls here: https://github.com/apache/datafusion/issues/12904 I looked at making this change directly in `aggregate_test_100.csv` (adding the two null columns) but there are 75 tests tightly coupled to the format of the csv. If the data duplication is a concern, I think it would still be better to duplicate temporarily and incrementally updates tests to use the new `aggregate_test_100_with_nulls`. Then when the test logic is fixed reduce to a single testing csv. -- 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: issues-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
