Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
-
None
Description
The new BatchTranslator has this method:
List<BatchParameterBinding> createBindings(BatchQueryRow row);
The list contents differ from row to row (e.g. due to NULL handling). This means we need to create a new List and new BatchParameterBindings, which results in an inefficiency in the default mode of operation. This also makes crypto-related interception doubly inefficient, as we'll need to compile encryptors for each row individually. So need to batch this somehow. E.g. by allocating a single BatchParameterBinding[], filling it with bindings and then reusing it for each row. Excluded bindings for a given row will be marked with some flag. This will also allow to precompile encryptors in the crypto module.