All NDB read results (including those containing BLOB and TEXT columns)

are now ValueObjects.

Author: jdduncan

Adapter/impl/ndb/NdbOperation.js

@@ -732,11 +732,7 @@ function buildOperationResult(transactionHandler, op, op_ndb_error, execMode) {
     }
 
     if(op.result.success && op.opcode === opcodes.OP_READ) {
-//      if(op.tableHandler.numberOfLobColumns) {
-//        readResultRow(op);    // Objects with BLOBs get plain JS Objects
-//      } else {
-        buildValueObject(op); // Objects without BLOBS get ValueObjects
-//      }
+      buildValueObject(op);
     } 
   }
   if(udebug.is_detail()) udebug.log("buildOperationResult finished:", op.result);

Adapter/impl/ndb/src/NdbTypeEncoders.cpp

@@ -1177,11 +1177,6 @@ Handle<Value> CharWriter(const NdbDictionary::Column * col,
   return writerOK;
 }
 
-// TODO:  In both CHAR and VARCHAR readers,
-// in the case where the result is not a ValueObject, but the string is 
-// externalized from an ASCII or UTF16LE buffer, verify that the buffer
-// contents will remain valid for the whole lifetime of the string.
- 
 // Templated encoder for Varchar and LongVarchar
 template<typename LENGTHTYPE>
 Handle<Value> varcharReader(const NdbDictionary::Column *col,