Acknowledge method in SQLService is always called

[nhuray]

The acknowledge method in SQLService is always called even if the rivertable parameter is set to false (Managing updates with Elasticsearch versioning).

Indeed the BulkOperation is always created with a BulkAcknowledge :

 operation = new BulkOperation(client, logger).setIndex(indexName).setType(typeName).setVersioning(versioning)
                .setBulkSize(bulkSize).setMaxActiveRequests(maxBulkRequests)
                .setMillisBeforeContinue(bulkTimeout.millis()).setAcknowledge(riverName.getName(), service);

we have to replace it by this code :

 operation = new BulkOperation(client, logger).setIndex(indexName).setType(typeName).setVersioning(versioning)
                .setBulkSize(bulkSize).setMaxActiveRequests(maxBulkRequests)
                .setMillisBeforeContinue(bulkTimeout.millis()).setAcknowledge(riverName.getName(), rivertable ? service : null);

[jprante]

Thank you, good catch!