Odin-BN opened a new pull request, #778:
URL: https://github.com/apache/arrow-java/pull/778

   ## What's Changed
   
   This PR improves error handling in 'CompositeJdbConsumer' by safely handling 
null 'ValueVector' and 'ResultSet' values to prevent 'NullPointerException', 
and return the intended 'JdbcConsumerException'.
   
   ### How was this tested? 
   
   Added two unit tests in 'CompositeJdbcConsumerTest':
   - 'testHandlesJdbcConsumerExceptionGracefully' verifies 'ArrowType' is 
captured if vector is valid, and returns the 'JdbcConsumerException' even 
though 'ResultSet' is null (which gave another NullPointerException previously).
   - 'testJdbcConsumerExceptionWhenArrowTypeIsNull' verifies 'ArrowType' is 
'null' and returns the 'JdbcConsumerException'.
   
   All tests pass after the change.
   
   Closes #734 
   


-- 
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]

Reply via email to