Remove unscoped markers

Author: jhawthorn

lib/vernier/result.rb

@@ -26,13 +26,6 @@ def weights; threads.values.flat_map { _1[:weights] }; end
     def samples; threads.values.flat_map { _1[:samples] }; end
     def sample_categories; threads.values.flat_map { _1[:sample_categories] }; end
 
-    # Legacy format. Avoid
-    def markers
-      threads&.flat_map do |tid, thread|
-        thread[:markers] || []
-      end || []
-    end
-
     # Realtime in nanoseconds since the unix epoch
     def started_at
       started_at_mono_ns = meta[:started_at]

test/test_active_support.rb

@@ -19,7 +19,7 @@ def test_instrument
       ActiveSupport::Notifications.instrument("foo.bar") {}
     end
 
-    markers = result.markers.select{|x| x[1] == "foo.bar" }
+    markers = result.main_thread[:markers].select{|x| x[1] == "foo.bar" }
     assert_equal 1, markers.size
 
     marker = markers[0]
@@ -33,7 +33,7 @@ def test_instrument_without_block
       ActiveSupport::Notifications.instrument("foo.bar")
     end
 
-    markers = result.markers.select{|x| x[1] == "foo.bar" }
+    markers = result.main_thread[:markers].select{|x| x[1] == "foo.bar" }
     assert_equal 1, markers.size
 
     marker = markers[0]
@@ -47,7 +47,7 @@ def test_instrument_publish
       ActiveSupport::Notifications.publish("foo.bar")
     end
 
-    markers = result.markers.select{|x| x[1] == "foo.bar" }
+    markers = result.main_thread[:markers].select{|x| x[1] == "foo.bar" }
     assert_equal 0, markers.size
   end
 

test/test_time_collector.rb

@@ -13,7 +13,7 @@ def test_receives_gc_events
     assert_valid_result result
     # make sure we got all GC events (since we did GC.start twice)
     assert_equal ["GC end marking", "GC end sweeping", "GC pause", "GC start"].sort,
-      result.markers.map { |x| x[1] }.grep(/^GC/).uniq.sort
+      result.main_thread[:markers].map { |x| x[1] }.grep(/^GC/).uniq.sort
   end
 
   def test_time_collector