Skip to content

Commit

Permalink
[Profile] Test heap snapshot in child process (#47205)
Browse files Browse the repository at this point in the history 
Save memory (especially important for 32-bit) by running in a new
process, and not one that has already run a lot of other test. This
avoids a possible OOM situation.
  • Loading branch information
@apaz-cli
apaz-cli authored Oct 18, 2022
1 parent b55c15e commit bc7500b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion stdlib/Profile/test/runtests.jl
View file
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ end

@testset "HeapSnapshot" begin
fname = tempname()
Profile.take_heap_snapshot(fname)
run(`$(Base.julia_cmd()) --startup-file=no -e "using Profile; Profile.take_heap_snapshot($(repr(fname)))"`)

open(fname) do fs
@test readline(fs) != ""
Expand Down

2 comments on commit bc7500b

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Executing the daily package evaluation, I will reply here when finished:

@nanosoldier runtests(ALL, isdaily = true, configuration=(rr=true,))

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your package evaluation job has completed - possible new issues were detected. A full report can be found here.

Please sign in to comment.