Ensure getStatementInfo executes statement only when needed to

Author: sharadraju

lib/thin/protocol/messages/execute.js

@@ -275,7 +275,7 @@ class ExecuteMessage extends MessageWithData {
     // no rows have yet been sent so the header information needs to be sent
     if (this.currentRow === 0) {
       let stmt = this.statement;
-      if (stmt.cursorId !== 0 && !stmt.requiresFullExecute && !stmt.requiresDefine && !stmt.isDdl) {
+      if (stmt.cursorId !== 0 && !stmt.requiresFullExecute && !this.parseOnly && !stmt.requiresDefine && !stmt.isDdl && !this.batchErrors) {
         if (stmt.isQuery && !stmt.requiresDefine && this.options.prefetchRows > 0) {
           this.functionCode = constants.TNS_FUNC_REEXECUTE_AND_FETCH;
         } else {

test/getStmtInfo.js

@@ -471,4 +471,13 @@ describe('162. getStmtInfo.js', function() {
     assert.deepStrictEqual(result.rows[0], ['HELLO']);
   });
 
+  it('162.35 Parse and Execute twice ', async function() {
+    const sql = "insert into nodb_number (num, content) values (:num, :val)";
+    let key = 555;
+    for (let i = 0; i < 2; i++) {
+      await conn.getStatementInfo(sql);
+      await conn.execute(sql, {num:key, val:1234});
+      key = key + 1;
+    }
+  });
 });