Remove PLUGINS legacy code

Author: jpsamaroo

src/Dagger.jl

@@ -18,11 +18,6 @@ using Requires
 using MacroTools
 using TimespanLogging
 
-const PLUGINS = Dict{Symbol,Any}()
-const PLUGIN_CONFIGS = Dict{Symbol,String}(
-    :scheduler => "Dagger.Sch"
-)
-
 include("lib/util.jl")
 
 # Distributed data

src/compute.jl

@@ -20,10 +20,7 @@ runs the scheduler with the specified options. Returns a Chunk which references
 the result.
 """
 function compute(ctx::Context, d::Thunk; options=nothing)
-    scheduler = get!(PLUGINS, :scheduler) do
-        get_type(PLUGIN_CONFIGS[:scheduler])
-    end
-    res = scheduler.compute_dag(ctx, d; options=options)
+    result = Sch.compute_dag(ctx, d; options=options)
     if ctx.log_file !== nothing
         if ctx.log_sink isa TimespanLogging.LocalEventLog
             logs = TimespanLogging.get_logs!(ctx.log_sink)
@@ -34,12 +31,12 @@ function compute(ctx::Context, d::Thunk; options=nothing)
             @warn "Context log_sink not set to LocalEventLog, skipping"
         end
     end
-    res
+    result
 end
 
 function debug_compute(ctx::Context, args...; profile=false, options=nothing)
-    @time res = compute(ctx, args...; options=options)
-    get_logs!(ctx.log_sink), res
+    @time result = compute(ctx, args...; options=options)
+    get_logs!(ctx.log_sink), result
 end
 
 function debug_compute(arg; profile=false, options=nothing)
@@ -53,11 +50,7 @@ Base.@deprecate gather(x) collect(x)
 
 cleanup() = cleanup(Context(global_context()))
 function cleanup(ctx::Context)
-    if :scheduler in keys(PLUGINS)
-        scheduler = PLUGINS[:scheduler]
-        (scheduler).cleanup(ctx)
-        delete!(PLUGINS, :scheduler)
-    end
+    Sch.cleanup(ctx)
     nothing
 end