[ARROW-17199] [FlightRPC][C++][Java] Fix example Flight SQL servers

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 10.0.0
• Component/s: C++, FlightRPC, Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/32491
• Assignee: David Li
• Reporter: David Li
• Created: 25/Jul/22 17:37
• Updated: 11/Jan/23 11:49
• Resolved: 04/Aug/22 12:54

Description

There are a number of small bugs in the Java Flight SQL example (e.g. binding parameters to the wrong index, not handling null parameter values, not properly reporting errors, things throwing SQLException that should use Arrow/general Java exceptions) that should be fixed.

Also, CommandPreparedStatementQuery is implemented wrong in both C++ and Java servers: only the last parameter value will be bound/executed when executing a prepared statement that returns a result set. But this contradicts the spec, which states "All of the bound parameter sets will be executed as a single atomic execution". The server(s) need to cache the uploaded Arrow data and execute using all input rows.

Issue Links

• depends upon: ARROW-17004 [Java] Implement Arrow->JDBC prepared statement parameters for arrow-jdbc
• links to: GitHub Pull Request #13710