CAE-193: add "IGNORE" options to time series commands (for v5 branch)

packages/time-series/lib/commands/ADD.spec.ts

@@ -57,14 +57,43 @@ describe('TS.ADD', () => {
       );
     });
 
+    it ('with IGNORE', () => {
+      it('no values', () => {
+        assert.deepEqual(
+          ADD.transformArguments('key', '*', 1, {
+            IGNORE: { }
+          }),
+          ['TS.ADD', 'key', '*', '1', 'IGNORE', '0', '0']
+        )
+      });
+
+      it('with MAX_TIME_DIFF', () => {
+        assert.deepEqual(
+          ADD.transformArguments('key', '*', 1, {
+            IGNORE: { MAX_TIME_DIFF: 1}
+          }),
+          ['TS.ADD', 'key', '*', '1', 'IGNORE', '1', '0']
+        )
+      });
+
+      it('with MAX_VAL_DIFF', () => {
+        assert.deepEqual(
+          ADD.transformArguments('key', '*', 1, {
+            IGNORE: { MAX_VAL_DIFF: 1}
+          }),
+          ['TS.ADD', 'key', '*', '1', 'IGNORE', '0', '1']
+        )
+      });
+    });
+
     it('with RETENTION, ENCODING, CHUNK_SIZE, ON_DUPLICATE, LABELS', () => {
       assert.deepEqual(
         ADD.transformArguments('key', '*', 1, {
           RETENTION: 1,
           ENCODING: TIME_SERIES_ENCODING.UNCOMPRESSED,
           CHUNK_SIZE: 1,
           ON_DUPLICATE: TIME_SERIES_DUPLICATE_POLICIES.BLOCK,
-          LABELS: { label: 'value' }
+          LABELS: { label: 'value' },
         }),
         ['TS.ADD', 'key', '*', '1', 'RETENTION', '1', 'ENCODING', 'UNCOMPRESSED', 'CHUNK_SIZE', '1', 'ON_DUPLICATE', 'BLOCK', 'LABELS', 'label', 'value']
       );

packages/time-series/lib/commands/ADD.ts

@@ -8,15 +8,22 @@ import {
   TimeSeriesDuplicatePolicies,
   Labels,
   pushLabelsArgument,
-  Timestamp
+  Timestamp,
+  pushIgnoreArgument
 } from '.';
 
+export interface TsIgnoreOptions {
+  maxTimeDiff: number;
+  maxValDiff: number;
+}
+
 export interface TsAddOptions {
   RETENTION?: number;
   ENCODING?: TimeSeriesEncoding;
   CHUNK_SIZE?: number;
   ON_DUPLICATE?: TimeSeriesDuplicatePolicies;
   LABELS?: Labels;
+  IGNORE?: TsIgnoreOptions;
 }
 
 export default {
@@ -47,6 +54,8 @@ export default {
 
     pushLabelsArgument(args, options?.LABELS);
 
+    pushIgnoreArgument(args, options?.IGNORE);
+
     return args;
   },
   transformReply: undefined as unknown as () => NumberReply

packages/time-series/lib/commands/ALTER.spec.ts

@@ -48,13 +48,44 @@ describe('TS.ALTER', () => {
       );
     });
 
+    it('with IGNORE', () => {
+      testUtils.isVersionGreaterThanHook([7, 4]);
+
+      it('no values', () => {
+        assert.deepEqual(
+          ALTER.transformArguments('key', {
+            IGNORE: { }
+          }),
+          ['TS.ALTER', 'key', 'IGNORE', '0', '0']
+        )
+      });
+
+      it('with MAX_TIME_DIFF', () => {
+        assert.deepEqual(
+          ALTER.transformArguments('key', {
+            IGNORE: { MAX_TIME_DIFF: 1}
+          }),
+          ['TS.ALTER', 'key', 'IGNORE', '1', '0']
+        )
+      });
+
+      it('with MAX_VAL_DIFF', () => {
+        assert.deepEqual(
+          ALTER.transformArguments('key', {
+            IGNORE: { MAX_VAL_DIFF: 1}
+          }),
+          ['TS.ALTER', 'key', 'IGNORE', '0', '1']
+        )
+      });
+    });
+
     it('with RETENTION, CHUNK_SIZE, DUPLICATE_POLICY, LABELS', () => {
       assert.deepEqual(
         ALTER.transformArguments('key', {
           RETENTION: 1,
           CHUNK_SIZE: 1,
           DUPLICATE_POLICY: TIME_SERIES_DUPLICATE_POLICIES.BLOCK,
-          LABELS: { label: 'value' }
+          LABELS: { label: 'value' },
         }),
         ['TS.ALTER', 'key', 'RETENTION', '1', 'CHUNK_SIZE', '1', 'DUPLICATE_POLICY', 'BLOCK', 'LABELS', 'label', 'value']
       );

packages/time-series/lib/commands/ALTER.ts

@@ -1,8 +1,8 @@
 import { RedisArgument, SimpleStringReply, Command } from '@redis/client/dist/lib/RESP/types';
 import { TsCreateOptions } from './CREATE';
-import { pushRetentionArgument, pushChunkSizeArgument, pushDuplicatePolicy, pushLabelsArgument } from '.';
+import { pushRetentionArgument, pushChunkSizeArgument, pushDuplicatePolicy, pushLabelsArgument, pushIgnoreArgument } from '.';
 
-export type TsAlterOptions = Pick<TsCreateOptions, 'RETENTION' | 'CHUNK_SIZE' | 'DUPLICATE_POLICY' | 'LABELS'>;
+export type TsAlterOptions = Pick<TsCreateOptions, 'RETENTION' | 'CHUNK_SIZE' | 'DUPLICATE_POLICY' | 'LABELS' | 'IGNORE'>;
 
 export default {
   FIRST_KEY_INDEX: 1,
@@ -18,6 +18,8 @@ export default {
 
     pushLabelsArgument(args, options?.LABELS);
 
+    pushIgnoreArgument(args, options?.IGNORE);
+
     return args;
   },
 transformReply: undefined as unknown as () => SimpleStringReply<'OK'>

packages/time-series/lib/commands/CREATE.spec.ts

@@ -57,14 +57,45 @@ describe('TS.CREATE', () => {
       );
     });
 
+    it('with IGNORE', () => {
+      testUtils.isVersionGreaterThanHook([7, 4]);
+
+      it('no values', () => {
+        assert.deepEqual(
+          CREATE.transformArguments('key', {
+            IGNORE: { }
+          }),
+          ['TS.CREATE', 'key', 'IGNORE', '0', '0']
+        )
+      });
+
+      it('with MAX_TIME_DIFF', () => {
+        assert.deepEqual(
+          CREATE.transformArguments('key', {
+            IGNORE: { MAX_TIME_DIFF: 1}
+          }),
+          ['TS.CREATE', 'IGNORE', '1', '0']
+        )
+      });
+
+      it('with MAX_VAL_DIFF', () => {
+        assert.deepEqual(
+          CREATE.transformArguments('key', {
+            IGNORE: { MAX_VAL_DIFF: 1}
+          }),
+          ['TS.CREATE', 'IGNORE', '0', '1']
+        )
+      });
+    });
+
     it('with RETENTION, ENCODING, CHUNK_SIZE, DUPLICATE_POLICY, LABELS', () => {
       assert.deepEqual(
         CREATE.transformArguments('key', {
           RETENTION: 1,
           ENCODING: TIME_SERIES_ENCODING.UNCOMPRESSED,
           CHUNK_SIZE: 1,
           DUPLICATE_POLICY: TIME_SERIES_DUPLICATE_POLICIES.BLOCK,
-          LABELS: { label: 'value' }
+          LABELS: { label: 'value' },
         }),
         ['TS.CREATE', 'key', 'RETENTION', '1', 'ENCODING', 'UNCOMPRESSED', 'CHUNK_SIZE', '1', 'DUPLICATE_POLICY', 'BLOCK', 'LABELS', 'label', 'value']
       );

packages/time-series/lib/commands/CREATE.ts

@@ -7,15 +7,18 @@ import {
   TimeSeriesDuplicatePolicies,
   pushDuplicatePolicy,
   Labels,
-  pushLabelsArgument
+  pushLabelsArgument,
+  pushIgnoreArgument
 } from '.';
+import { TsIgnoreOptions } from './ADD';
 
 export interface TsCreateOptions {
   RETENTION?: number;
   ENCODING?: TimeSeriesEncoding;
   CHUNK_SIZE?: number;
   DUPLICATE_POLICY?: TimeSeriesDuplicatePolicies;
   LABELS?: Labels;
+  IGNORE?: TsIgnoreOptions;
 }
 
 export default {
@@ -34,6 +37,8 @@ export default {
 
     pushLabelsArgument(args, options?.LABELS);
 
+    pushIgnoreArgument(args, options?.IGNORE);
+
     return args;
   },
   transformReply: undefined as unknown as () => SimpleStringReply<'OK'>

packages/time-series/lib/commands/DECRBY.spec.ts

@@ -56,6 +56,37 @@ describe('TS.DECRBY', () => {
       );
     });
 
+    it ('with IGNORE', () => {
+      testUtils.isVersionGreaterThanHook([7, 4]);
+
+      it('no values', () => {
+        assert.deepEqual(
+          DECRBY.transformArguments('key', 1, {
+            IGNORE: { }
+          }),
+          ['TS.DECRBY', 'key', '1', 'IGNORE', '0', '0']
+        )
+      });
+
+      it('MAX_TIME_DIFF', () => {
+        assert.deepEqual(
+          DECRBY.transformArguments('key', 1, {
+            IGNORE: { MAX_TIME_DIFF: 1}
+          }),
+          ['TS.DECRBY', 'key', '1', 'IGNORE', '1', '0']
+        )
+      });
+
+      it('MAX_VAL_DIFF', () => {
+        assert.deepEqual(
+          DECRBY.transformArguments('key', 1, {
+            IGNORE: { MAX_VAL_DIFF: 1}
+          }),
+          ['TS.DECRBY', 'key', '1', 'IGNORE', '0', '1']
+        )
+      });
+    });
+
     it('with TIMESTAMP, RETENTION, UNCOMPRESSED, CHUNK_SIZE and LABELS', () => {
       assert.deepEqual(
         DECRBY.transformArguments('key', 1, {

packages/time-series/lib/commands/INCRBY.spec.ts

@@ -65,6 +65,37 @@ describe('TS.INCRBY', () => {
       );
     });
 
+    it ('with IGNORE', () => {
+      testUtils.isVersionGreaterThanHook([7, 4]);
+
+      it('no values', () => {
+        assert.deepEqual(
+          INCRBY.transformArguments('key', 1, {
+            IGNORE: { }
+          }),
+          ['TS.INCRBY', 'key', '1', 'IGNORE', '0', '0']
+        )
+      });
+
+      it('MAX_TIME_DIFF', () => {
+        assert.deepEqual(
+          INCRBY.transformArguments('key', 1, {
+            IGNORE: { MAX_TIME_DIFF: 1}
+          }),
+          ['TS.INCRBY', 'key', '1', 'IGNORE', '1', '0']
+        )
+      });
+
+      it('MAX_VAL_DIFF', () => {
+        assert.deepEqual(
+          INCRBY.transformArguments('key', 1, {
+            IGNORE: { MAX_VAL_DIFF: 1}
+          }),
+          ['TS.INCRBY', 'key', '1', 'IGNORE', '0', '1']
+        )
+      });
+    });
+
     it('with TIMESTAMP, RETENTION, UNCOMPRESSED, CHUNK_SIZE and LABELS', () => {
       assert.deepEqual(
         INCRBY.transformArguments('key', 1, {

packages/time-series/lib/commands/INCRBY.ts

@@ -1,12 +1,14 @@
 import { RedisArgument, NumberReply, Command } from '@redis/client/dist/lib/RESP/types';
-import { Timestamp, transformTimestampArgument, pushRetentionArgument, pushChunkSizeArgument, Labels, pushLabelsArgument } from '.';
+import { Timestamp, transformTimestampArgument, pushRetentionArgument, pushChunkSizeArgument, Labels, pushLabelsArgument, pushIgnoreArgument } from '.';
+import { TsIgnoreOptions } from './ADD';
 
 export interface TsIncrByOptions {
   TIMESTAMP?: Timestamp;
   RETENTION?: number;
   UNCOMPRESSED?: boolean;
   CHUNK_SIZE?: number;
   LABELS?: Labels;
+  IGNORE?: TsIgnoreOptions;
 }
 
 export function transformIncrByArguments(
@@ -35,6 +37,8 @@ export function transformIncrByArguments(
 
   pushLabelsArgument(args, options?.LABELS);
 
+  pushIgnoreArgument(args, options?.IGNORE);
+
   return args;
 }
 

packages/time-series/lib/commands/index.ts

@@ -1,5 +1,5 @@
 import type { BlobStringReply, CommandArguments, DoubleReply, NumberReply, RedisArgument, RedisCommands, TuplesReply, UnwrapReply } from '@redis/client/dist/lib/RESP/types';
-import ADD from './ADD';
+import ADD, { TsIgnoreOptions } from './ADD';
 import ALTER from './ALTER';
 import CREATE from './CREATE';
 import CREATERULE from './CREATERULE';
@@ -67,6 +67,12 @@ export default {
   revRange: REVRANGE
 } as const satisfies RedisCommands;
 
+export function pushIgnoreArgument(args: Array<RedisArgument>, ignore?: TsIgnoreOptions) {
+  if (ignore !== undefined) {
+    args.push('IGNORE', ignore.maxTimeDiff.toString(), ignore.maxValDiff.toString());
+  }
+}
+
 export function pushRetentionArgument(args: Array<RedisArgument>, retention?: number) {
   if (retention !== undefined) {
     args.push('RETENTION', retention.toString());