Enhance NoSQLHandleConfig.set_logger to accept None logger to disable logging.

Author: pengfei0107

src/borneo/common.py

@@ -185,7 +185,7 @@ def check_list(data, name):
 
     @staticmethod
     def check_logger(data, name):
-        if not isinstance(data, Logger):
+        if data is not None and not isinstance(data, Logger):
             raise IllegalArgumentException(name + ' must be a Logger.')
 
     @staticmethod

src/borneo/config.py

@@ -561,6 +561,7 @@ def __init__(self, endpoint=None, provider=None):
         self._ssl_ctx = None
         self._ssl_protocol = None
         self._logger = None
+        self._is_default_logger = True
 
     def get_service_url(self):
         """
@@ -1114,14 +1115,15 @@ def set_logger(self, logger):
         """
         Sets the logger used for the driver.
 
-        :param logger: the logger.
+        :param logger: the logger or None, None means disable logging.
         :type logger: Logger
         :returns: self.
         :raises IllegalArgumentException: raises the exception if logger is not
             an instance of Logger.
         """
         CheckValue.check_logger(logger, 'logger')
         self._logger = logger
+        self._is_default_logger = False
         return self
 
     def get_logger(self):
@@ -1151,6 +1153,10 @@ def clone(self):
         self._auth_provider = auth_provider
         return clone_config
 
+    def is_default_logger(self):
+        # Internal use only
+        return self._is_default_logger
+
     #
     # Return a url from an endpoint string that has the format
     # [protocol:][//]host[:port]

src/borneo/driver.py

@@ -649,15 +649,15 @@ def _get_logger(self, config):
         Returns the logger used for the driver. If no logger is specified,
         create one based on this class name.
         """
-        if config.get_logger() is not None:
-            logger = config.get_logger()
-        else:
+        if config.get_logger() is None and config.is_default_logger():
             logger = getLogger(self.__class__.__name__)
             logger.setLevel(WARNING)
             log_dir = path.join(path.abspath(path.dirname(argv[0])), 'logs')
             if not path.exists(log_dir):
                 mkdir(log_dir)
             logger.addHandler(FileHandler(path.join(log_dir, 'driver.log')))
+        else:
+            logger = config.get_logger()
         return logger
 
     @staticmethod