Skip to content

Commit

Permalink
[Data] Remove unnecessary setting of global logging level to INFO whe…
Browse files Browse the repository at this point in the history
…n using Ray Data (#34347)

When initializing Ray Data, the global logging level is set to `INFO`, which causes non-Ray `INFO` logs to be unintentionally emitted (the default level in the `logging` library is `WARNING`, which would normally ignore `INFO`-level logs). We remove an unnecessary setting of the logging level in `DatasetLogger` which resolves this issue.

Signed-off-by: Scott Lee <[email protected]>
  • Loading branch information
scottjlee authored Apr 13, 2023
1 parent 120e059 commit 10d0b8a
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions python/ray/data/_internal/dataset_logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,6 @@ def _initialize_logger(self) -> logging.Logger:
for writing to the Dataset log file. Not intended (nor should it be necessary)
to call explicitly. Assumes that `ray.init()` has already been called prior
to calling this method; otherwise raises a `ValueError`."""
# In the case where logger has not yet been set up in another file,
# initialize basic configs to set log level and format.
logging.basicConfig(
level=LOGGER_LEVEL.upper(),
format=LOGGER_FORMAT,
)

# We initialize a logger using the given base `log_name`, which
# logs to stdout. Logging with this logger to stdout is enabled by the
Expand Down

0 comments on commit 10d0b8a

Please sign in to comment.