node-s3tables 0.0.7 → 0.0.9

package/dist/index.d.ts

@@ -87,7 +87,7 @@ interface IcebergMetadata {
     'default-spec-id': number;
     'partition-specs': IcebergPartitionSpec[];
     'last-partition-id': number;
-    'current-snapshot-id'?: bigint | number;
+    'current-snapshot-id': bigint | number;
     location: string;
 }
 
@@ -171,6 +171,7 @@ interface AddDataFilesParams {
     tableBucketARN: string;
     namespace: string;
     name: string;
+    snapshotId?: bigint;
     lists: AddFileList[];
     retryCount?: number;
 }

package/dist/index.js

@@ -745,16 +745,16 @@ function _encodeValue(raw, transform, out_type) {
     }
 }
 const NaNValue = NaN;
-function makeBounds(paritions, spec, schema) {
+function makeBounds(partitions, spec, schema) {
     return spec.fields.map((f) => {
         const schemaField = schema.fields.find((sf) => sf.id === f['source-id']);
         if (!schemaField) {
             throw new Error(`Schema field not found for source-id ${f['source-id']}`);
         }
-        if (!(f.name in paritions)) {
-            throw new Error(`paritions missing ${f.name}`);
+        if (!(f.name in partitions)) {
+            throw new Error(`partitions missing ${f.name}`);
         }
-        const raw = paritions[f.name];
+        const raw = partitions[f.name];
         if (typeof raw === 'number' && isNaN(raw)) {
             return NaNValue;
         }
@@ -1177,12 +1177,11 @@ async function addDataFiles(params) {
     if (!region) {
         throw new Error('bad tableBucketARN');
     }
-    const snapshot_id = _randomBigInt64();
+    const snapshot_id = params.snapshotId ?? _randomBigInt64();
     const metadata = await getMetadata(params);
     const bucket = metadata.location.split('/').slice(-1)[0];
-    const parent_snapshot_id = BigInt(metadata['current-snapshot-id'] ?? -1n);
-    const snapshot = metadata.snapshots.find((s) => s['snapshot-id'] === parent_snapshot_id) ??
-        null;
+    const parent_snapshot_id = BigInt(metadata['current-snapshot-id']);
+    const snapshot = metadata.snapshots.find((s) => BigInt(s['snapshot-id']) === parent_snapshot_id) ?? null;
     if (!bucket) {
         throw new Error('bad manifest location');
     }
@@ -1313,7 +1312,7 @@ async function addDataFiles(params) {
         }
         // we do a merge in the append only simultanious case
         const conflict_metadata = await getMetadata(params);
-        const conflict_snapshot_id = BigInt(conflict_metadata['current-snapshot-id'] ?? -1n);
+        const conflict_snapshot_id = BigInt(conflict_metadata['current-snapshot-id']);
         if (conflict_snapshot_id <= 0n) {
             throw new Error('conflict');
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-s3tables",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "description": "node api for dealing with s3tables",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",