[API improvement] Types of params of PreparedStatement in Query or Update

[BillyYccc]

Status

Currently query or update in PgOperations use List<Object> to hold multiple params, API is something like this void query(String sql, List<Object> params, Handler<AsyncResult<ResultSet>> handler);
or void update(String sql, List<Object> params, Handler<AsyncResult<UpdateResult>> handler);

Motivation

As it's defined in vertx-sql-common, SQLOperations use JsonArray to hold mutiple params like SQLOperations queryWithParams(String sql, JsonArray params, Handler<AsyncResult<ResultSet>> resultHandler);
or SQLOperations updateWithParams(String sql, JsonArray params, Handler<AsyncResult<UpdateResult>> resultHandler);
It's better to keep consistency, is it right?

Apart from this, List<Object> is also not supported by Codegen when passing as input parameters, so the method has to be ignored by @GenIgnore. When using Rxi-fied API, there is no methods to take multiple params as one collection, users have to maintain a state of how many params they use and then choose which method to use, this is not convenient when facing dynamic SQLs.

Therefore is there any possibility to change this?

[vietj]

I think we should work out in vertx the ability to support List<Object> and use it

[vietj]

@BillyYccc the master has been updated to use a Tuple object that is now available in RxJava as well, would you mind to try it ?

[BillyYccc]

@vietj Thanks for your nicely effort, This works now!