cadence/out-of-your-element
cadence/out-of-your-element
4
10
Fork 7
Code Issues 15 Pull requests 3 Releases 29 Activity

Various error log improvements #100

Open
l1s5 wants to merge 6 commits from l1s5/out-of-your-element:log-improvements into main
pull from: l1s5/out-of-your-element:log-improvements
merge into: cadence:main
Conversation 2 Commits 6 Files changed 5 +4 -1
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit d5972187eb - Show all commits

Log d2m retrigger timeouts

Prev Next
l1s5 2026-10-25 21:31:27 +09:00
commit d5972187eb
Signed by untrusted user: l1s5
GPG key ID: D0A90CC905B58D9B

5
src/d2m/actions/retrigger.js
View file

@ -38,7 +38,10 @@ const storage = sync.remember(() => new class {
this.resolves.set(id, (this.resolves.get(id) || []).concat(resolve)) // add to list in map value this.resolves.set(id, (this.resolves.get(id) || []).concat(resolve)) // add to list in map value
if (!this.timers.has(id)) { if (!this.timers.has(id)) {
debugRetrigger(`[retrigger] SET TIMER id = ${id}`) debugRetrigger(`[retrigger] SET TIMER id = ${id}`)
this.timers.set(id, setTimeout(() => this.resolve(id, false), 60 * 1000).unref()) // 1 minute this.timers.set(id, setTimeout(() => {
console.warn(`[retrigger] gave up waiting for ${id} to finish bridging after 1 minute, skipping ${this.resolves.get(id)?.length ?? 0} pending changes`)
cadence commented 2026-10-26 13:15:18 +00:00
Copy link

I don't want to print retrigger mismatches since these are expected in normal operation. Logging this is noisy and wouldn't help to solve actual problems in retrigger. The complete logging from DEBUG_RETRIGGER is necessary for that.

I don't want to print retrigger mismatches since these are expected in normal operation. Logging this is noisy and wouldn't help to solve actual problems in retrigger. The complete logging from DEBUG_RETRIGGER is necessary for that.
this.resolve(id, false)
}, 60 * 1000).unref()) // 1 minute
} }
} }