graham/prop
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: catch sandbox pool exits in analyze_all background Task
Some checks failed
Build and Push / Build and Push Docker Image (push) Has been cancelled
Details

Browse source 
The ANALYZE queries run via Task.Supervised which doesn't hold
sandbox ownership. rescue only catches exceptions; sandbox-pool
checkout failures come as exit signals. Add catch :exit so the
Task logs a warning instead of crashing.
This commit is contained in:
Graham McIntire 2026-08-05 08:32:05 -05:00
parent 32818d5866
commit 56b724b5a4
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
lib/microwaveprop/weather.ex
View file

@ -197,6 +197,9 @@ defmodule Microwaveprop.Weather do
Logger.info("Weather.analyze_all: ANALYZE #{t} done in #{elapsed}ms") Logger.info("Weather.analyze_all: ANALYZE #{t} done in #{elapsed}ms")
rescue rescue
e -> Logger.warning("Weather.analyze_all: ANALYZE #{t} failed: #{inspect(e)}") e -> Logger.warning("Weather.analyze_all: ANALYZE #{t} failed: #{inspect(e)}")
catch
:exit, reason ->
Logger.warning("Weather.analyze_all: ANALYZE #{t} sandbox exit: #{inspect(reason)}")
end end
end) end)