Skip to content

Commit 0f5e7a6

Browse files
committed
Update requires in LoggerThreadSafeLevel
The `concurrent` require was [added][1] previously because a `Concurrent::Map` was added to hold all of the log levels by `Thread` id. However, the `Map` was later [removed][2] by storing the log levels in the `Thread` locals (and later in `IsolatedExecutionState`) instead. The new implementation additionally removed the `cattr_accessor` (removing the need to require the "attribute_accessors" core_ext) and also replaced the [usage][3] of `Fiber` with `Thread` (so the require for `Fiber` is also no longer necessary). Since `Concurrent`, `Fiber`, and `cattr_accessor` are no longer used here, we can remove the requires. Since `Logger` is used here (but was previously required by `concurrent`), a require was added for it. [1]: 629efb6 [2]: 2379bc5 [3]: 56ec504
1 parent d8391b2 commit 0f5e7a6

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

activesupport/lib/active_support/logger_thread_safe_level.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
# frozen_string_literal: true
22

33
require "active_support/concern"
4-
require "active_support/core_ext/module/attribute_accessors"
5-
require "concurrent"
6-
require "fiber"
4+
require "logger"
75

86
module ActiveSupport
97
module LoggerThreadSafeLevel # :nodoc:

0 commit comments

Comments
 (0)