Skip to content
Snippets Groups Projects

lua worker.bg_worker: also report missing cqueues

Merged Vladimír Čunát requested to merge missing-cqueues into master
All threads resolved!

Otherwise people could get confusing errors like:

attempt to index field 'bg_worker' (a nil value)

Fixes #562 (closed) (kind-of)

At least in prefill case the error unfortunately won't show that it was "caused" by the prefill module, but perhaps that's OK-ish.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Vladimír Čunát resolved all threads

    resolved all threads

  • added 1 commit

    • d24b3ed0 - Apply suggestion to daemon/lua/sandbox.lua.in

    Compare with previous version

  • added 2 commits

    • eca53f7a - lua worker: improve message when missing cqueues
    • 030559a2 - lua worker.bg_worker: also report missing cqueues

    Compare with previous version

  • added 2 commits

    • bd6ec06d - lua worker.bg_worker: also report missing cqueues
    • ecc322df - lua worker: improve message when missing cqueues

    Compare with previous version

  • mentioned in commit a9ff9b8a

  • Please register or sign in to reply
    Loading