@empyria/restate 0.4.9 → 0.4.11

package/lib/Cron.js

@@ -11,12 +11,17 @@ export const setJobStateName = (jobStateName) => {
 }
 
 /**
- * A retry policy that never gives up: it backs off to one attempt a minute and keeps trying. Restate's default
- * stops after a few attempts and *pauses* the invocation, and a paused invocation of a virtual object holds the
- * object's lock, so every later call queues behind it for ever. A schedule that was due while the services were
- * down (a restart, a crash, a night off) must simply run when they are back, so jobs use this.
+ * A retry policy that practically never gives up: it backs off to one attempt a minute and keeps trying. Left to
+ * itself Restate stops after 70 attempts (about an hour at this pace) and *pauses* the invocation, and a paused
+ * invocation of a virtual object holds the object's lock, so every later call queues behind it for ever. A schedule
+ * that was due while the services were down (a restart, a crash, a night off) must simply run when they are back, so
+ * jobs use this.
+ *
+ * The attempt limit is the largest the server accepts. Leaving `maxAttempts` out, or setting it to `Infinity`, does
+ * not mean unlimited: the server then applies its own default of 70.
  */
 export const RETRY_FOREVER = {
+	maxAttempts: 2_147_483_647,
 	initialInterval: { seconds: 1 },
 	maxInterval: { minutes: 1 },
 	exponentiationFactor: 2,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@empyria/restate",
-	"version": "0.4.9",
+	"version": "0.4.11",
 	"description": "Restate.dev helpers for the Empyria nanoservice framework",
 	"license": "MIT",
 	"author": "Imre Fazekas <imre.fazekas@icloud.com>",
@@ -27,7 +27,7 @@
 	},
 	"dependencies": {
 		"@empyria/classification": "0.1.9",
-		"@empyria/common": "0.3.9",
+		"@empyria/common": "0.3.10",
 		"@restatedev/pubsub": "0.3.2",
 		"@restatedev/pubsub-client": "0.3.2",
 		"@restatedev/restate-sdk": "1.17.2",

package/test/Cron.test.js

@@ -293,9 +293,10 @@ describe('ensureCronJob', () => {
 })
 
 describe('retry policy', () => {
-	test('a job never gives up: no attempt limit, so it is never paused', async () => {
+	test('a job practically never gives up, so it is never paused', async () => {
 		const { RETRY_FOREVER } = await import('../lib/Cron.js')
-		expect(RETRY_FOREVER.maxAttempts).toBeUndefined()
+		// Omitting the limit (or Infinity) would leave the server default of 70 attempts.
+		expect(RETRY_FOREVER.maxAttempts).toBe(2_147_483_647)
 		expect(cronJob.options?.retryPolicy ?? cronJob.retryPolicy).toBeDefined()
 	})
 })