"Looking at this patch I noticed that there are two other possible calls which users might want to create the prepared statement:
prepareStatement(String sql, int columnIndexes)
prepareStatement(String sql, String columnNames)"
What would you see as the API for this? While I agree that those could be useful, I'm not sure it's really necessary:
1. Multi-column primary keys might be handled by the JDBC driver, if all columns are defined as being part of the primary key. This is pure speculation on my part.
2. Multi-column primary keys are fairly rare.
3. Whatever the API we come up with has to be replicated in the AsyncQueryRunner, so it's a lot of extra methods.
"Otherwise, the patch is missing (as noted above) JavaDocs and the async implementation, but looks sound. If you can update your patch with these additional things, I'll look to commit it. Thanks!"
Cool. I'll try to make time for it this week.