From 7e4966fbe51dac52dabba0444cc6e882f4a04ec3 Mon Sep 17 00:00:00 2001 From: alex lustenberg Date: Wed, 2 Dec 2020 13:59:04 -0500 Subject: [PATCH] increase log level of messages to trace (#5017) * cache/base - check start, stop, assumed unchanged if RO * command/base - assumed default * state - fetched * utils - assumed default * utils/fs - path inode --- dvc/cache/base.py | 10 +++++----- dvc/command/base.py | 2 +- dvc/state.py | 2 +- dvc/utils/__init__.py | 2 +- dvc/utils/fs.py | 2 +- 5 files changed, 9 insertions(+), 9 deletions(-) diff --git a/dvc/cache/base.py b/dvc/cache/base.py index a5f6b0b13..1f8ce26b9 100644 --- a/dvc/cache/base.py +++ b/dvc/cache/base.py @@ -106,7 +106,7 @@ class CloudCache: bool: True if data has changed, False otherwise. """ - logger.debug( + logger.trace( "checking if '%s'('%s') has changed.", path_info, hash_info ) @@ -134,7 +134,7 @@ class CloudCache: ) return True - logger.debug("'%s' hasn't changed.", path_info) + logger.trace("'%s' hasn't changed.", path_info) return False def link(self, from_info, to_info): @@ -345,14 +345,14 @@ class CloudCache: # Prefer string path over PathInfo when possible due to performance cache_info = self.hash_to_path(hash_info.value) if self.tree.is_protected(cache_info): - logger.debug( + logger.trace( "Assuming '%s' is unchanged since it is read-only", cache_info ) return False actual = self.tree.get_hash(cache_info) - logger.debug( + logger.trace( "cache '%s' expected '%s' actual '%s'", cache_info, hash_info, @@ -521,7 +521,7 @@ class CloudCache: failed = path_info elif not relink and not self.changed(path_info, hash_info): - logger.debug("Data '%s' didn't change.", path_info) + logger.trace("Data '%s' didn't change.", path_info) skip = True elif self.changed_cache( diff --git a/dvc/command/base.py b/dvc/command/base.py index e55471b4d..f1f2facb3 100644 --- a/dvc/command/base.py +++ b/dvc/command/base.py @@ -50,7 +50,7 @@ class CmdBase(ABC): """Default targets for `dvc repro`.""" from dvc.dvcfile import PIPELINE_FILE - logger.debug(f"assuming default target '{PIPELINE_FILE}'.") + logger.trace(f"assuming default target '{PIPELINE_FILE}'.") return [PIPELINE_FILE] @abstractmethod diff --git a/dvc/state.py b/dvc/state.py index 82f1f02bc..a7702066d 100644 --- a/dvc/state.py +++ b/dvc/state.py @@ -174,7 +174,7 @@ class State(StateBase): # pylint: disable=too-many-instance-attributes def _fetchall(self): ret = self.cursor.fetchall() - logger.debug("fetched: %s", ret) + logger.trace("fetched: %s", ret) return ret def _to_sqlite(self, num): diff --git a/dvc/utils/__init__.py b/dvc/utils/__init__.py index 2bceac938..e6a83ed35 100644 --- a/dvc/utils/__init__.py +++ b/dvc/utils/__init__.py @@ -454,7 +454,7 @@ def parse_target( return ret if is_valid_filename(target) else ret[::-1] if not path: - logger.debug("Assuming file to be '%s'", default) + logger.trace("Assuming file to be '%s'", default) return path or default, name diff --git a/dvc/utils/fs.py b/dvc/utils/fs.py index a5cdf1ca8..c3a47a4e0 100644 --- a/dvc/utils/fs.py +++ b/dvc/utils/fs.py @@ -26,7 +26,7 @@ def fs_copy(src, dst): def get_inode(path): inode = System.inode(path) - logger.debug("Path '%s' inode '%d'", path, inode) + logger.trace("Path '%s' inode '%d'", path, inode) return inode