[ARROW-7899] [Integration][Java] null type integration test

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 0.17.0
• Component/s: Integration, Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24122
• Assignee: Bryan Cutler
• Reporter: Neal Richardson
• Created: 20/Feb/20 20:22
• Updated: 11/Jan/23 07:56
• Resolved: 24/Feb/20 22:23

Description

From https://github.com/apache/arrow/pull/6368

 

lidavidm commented 2 days ago

https://github.com/apache/arrow/blob/master/java/tools/src/main/java/org/apache/arrow/tools/Integration.java#L208   If I'm not mistaken, this means that we only compare the data fully if there's actual data in both JSON and in the Arrow file? Though the Flight test is also potentially wrong:   https://github.com/apache/arrow/blob/master/java/flight/flight-core/src/main/java/org/apache/arrow/flight/example/integration/IntegrationTestClient.java#L166-L173 It only compares the last batch sent over the wire.

Issue Links

• blocks: ARROW-1636 [Format] Integration tests for null type
• relates to: ARROW-7933 [Java][Flight][Tests] Add roundtrip tests for Java Flight Test Client
• relates to: ARROW-7966 [Integration][Flight][C++] Client should verify each batch independently
• links to: GitHub Pull Request #6476