reuse old logger and give it a name

Signed-off-by: dentiny <[email protected]>

python/ray/_private/log_rotation_with_pipe.py

@@ -55,7 +55,7 @@ def open_pipe_with_rotation(
     lock = threading.Lock()
     cond = threading.Condition(lock)
 
-    logger = logging.getLogger()
+    logger = logging.getLogger(name=fname)
     logger.setLevel(logging.INFO)
     handler = RotatingFileHandler(
         fname, maxBytes=rotation_max_size, backupCount=rotation_file_num
@@ -88,8 +88,6 @@ def read_log_content_from_pipe():
     # Two threads are used here to make sure continuous read from pipe thus avoid
     # blocking write from application.
     def dump_log_content_to_buffer():
-        logger = logging.getLogger()
-
         while True:
             with cond:
                 while not log_content and not stopped:

python/ray/tests/test_logging.py

@@ -273,8 +273,6 @@ def component_file_only_one_log_entry(component):
             f"backup count {backup_count}, file count: {file_cnt}"
         )
 
-    # TODO(hjiang): Enable after log rotation implemented for user application.
-    #
     # Test application log, which starts with `worker-`.
     # Should be tested separately with other components since "worker" is a substring
     # of "python-core-worker".