Off By One Error in query/exector.go

[ayang64]

executeQuery() iterates over statements until each is processed or if an error is encountered that causes the loop to exit pre-maturely.

In that case (early exit) there is a loop that is meant send an appropriate response for any remaining statements (ErrNotExecuted).

// Send error results for any statements which were not executed.
for ; i < len(query.Statements)-1; i++ {
 if err := ctx.send(&Result{
  StatementID: i,
  Err:         ErrNotExecuted,
 }); err == ErrQueryAborted {
  return
 }
}

It looks like removing the -1 from the condition part of the for loop would work but i is sometimes set to the index of the previously successfully executed statement.

I've tried a few ways of fixing this but I think this requires a little more thought to fix properly.

The impact of this bug is that remaining statements aren't properly handled.