Use fixed length for dimensions count in vector images for Oracle Database 23ai

Author: sharadraju

lib/impl/datahandlers/constants.js

@@ -35,10 +35,8 @@ module.exports = {
   TNS_VECTOR_VERSION: 0,
 
   // vector flags
-  TNS_VECTOR_FLAG_DIM_UINT8: 0x0001,
-  TNS_VECTOR_FLAG_DIM_UINT32: 0x0002,
-  TNS_VECTOR_FLAG_NORMSRC: 0x0040,
-  TNS_VECTOR_FLAG_NORM: 0x0008,
+  TNS_VECTOR_FLAG_NORMSRC: 0x0010,
+  TNS_VECTOR_FLAG_NORM: 0x0002,
 
   // base JSON constants
   TNS_JSON_MAGIC_BYTE_1: 0xff,

lib/impl/datahandlers/vector.js

@@ -54,14 +54,7 @@ class VectorDecoder extends BaseBuffer {
       errors.throwErr(errors.ERR_VECTOR_VERSION_NOT_SUPPORTED, version);
     const flags = this.readUInt16BE();
     const vectorFormat = this.readUInt8();
-    let numElements;
-    if (flags & constants.TNS_VECTOR_FLAG_DIM_UINT8) {
-      numElements = this.readUInt8();
-    } else if (flags & constants.TNS_VECTOR_FLAG_DIM_UINT32) {
-      numElements = this.readUInt32BE();
-    } else {
-      numElements = this.readUInt16BE();
-    }
+    const numElements = this.readUInt32BE();
     let elementSize, result;
     if (vectorFormat === constants.VECTOR_FORMAT_FLOAT32) {
       elementSize = 4;
@@ -117,27 +110,15 @@ class VectorEncoder extends GrowableBuffer {
     }
 
     // Let server generate the norm (TNS_VECTOR_FLAG_NORMSRC)
-    let flags = constants.TNS_VECTOR_FLAG_NORM
+    const flags = constants.TNS_VECTOR_FLAG_NORM
       | constants.TNS_VECTOR_FLAG_NORMSRC;
-    const numElements = value.length;
-    if (numElements < 256) {
-      flags |= constants.TNS_VECTOR_FLAG_DIM_UINT8;
-    } else if (numElements > 65535) {
-      flags |= constants.TNS_VECTOR_FLAG_DIM_UINT32;
-    }
 
     // write header
     this.writeUInt8(constants.TNS_VECTOR_MAGIC_BYTE);
     this.writeUInt8(constants.TNS_VECTOR_VERSION);
     this.writeUInt16BE(flags);
     this.writeUInt8(vectorFormat);
-    if (numElements < 256) {
-      this.writeUInt8(numElements);
-    } else if (numElements < 65536) {
-      this.writeUInt16BE(numElements);
-    } else {
-      this.writeUInt32BE(numElements);
-    }
+    this.writeUInt32BE(value.length);
     this.reserveBytes(8);
 
     // write data