Closed
Description
It seems we have a regression in scope of SQL request completion after 9471340.
- here we get operation response code.
- Next line compares code status against 0 (success)
- If true the same code status is compared with EXECUTE (11) request code which always leads to be evaluated to false (0 never equals 11).
- As a result, completeSql method will never be called anymore.
protected void complete(TarantoolPacket packet, CompletableFuture<?> q) {
if (q != null) {
long code = packet.getCode();
if (code == 0) {
if (code == Code.EXECUTE.getId()) { // 0 == 11 always evaluates to false
completeSql(q, packet);
} else {
((CompletableFuture) q).complete(packet.getBody().get(Key.DATA.getId()));
}
} else {
Object error = packet.getBody().get(Key.ERROR.getId());
fail(q, serverError(code, error));
}
}
}