[ARROW-6218] [Java] Add UINT type test in integration to avoid potential overflow

• Type: Test
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22606
• Assignee: Ji Liu
• Reporter: Ji Liu
• Created: 13/Aug/19 03:15
• Updated: 11/Jan/23 07:45
• Resolved: 13/Aug/19 09:04

Description

As per discussion https://github.com/apache/arrow/pull/5002

For UINT type, when write/read json data in integration test, it extend data type(i.e. Long->BigInteger, Int->Long) to avoid potential overflow.

Like UINT8 the write side and read side code like this:

 

case UINT8:

  generator.writeNumber(UInt8Vector.getNoOverflow(buffer, index));

  break;

 

BigInteger value = parser.getBigIntegerValue();

buf.writeLong(value.longValue());

Should add a test to avoid potential overflow in the data transfer process.

Issue Links

• links to: GitHub Pull Request #5072