Merge pull request #142 from chamikaramj/remove_global_error_log_level

Updates encoding.py to not to set/reset global logger.

apitools/base/protorpclite/protojson.py

@@ -272,9 +272,6 @@ def __decode_dictionary(self, message_type, dictionary):
                 variant = self.__find_variant(value)
                 if variant:
                     message.set_unrecognized_field(key, value, variant)
-                else:
-                    logging.warning(
-                        'No variant found for unrecognized field: %s', key)
                 continue
 
             # This is just for consistency with the old behavior.

apitools/base/py/encoding.py

@@ -20,7 +20,6 @@
 import collections
 import datetime
 import json
-import logging
 import os
 import sys
 
@@ -276,16 +275,9 @@ def decode_message(self, message_type, encoded_message):
         if message_type in _CUSTOM_MESSAGE_CODECS:
             return _CUSTOM_MESSAGE_CODECS[
                 message_type].decoder(encoded_message)
-        # We turn off the default logging in protorpc. We may want to
-        # remove this later.
-        old_level = logging.getLogger().level
-        logging.getLogger().setLevel(logging.ERROR)
-        try:
-            result = _DecodeCustomFieldNames(message_type, encoded_message)
-            result = super(_ProtoJsonApiTools, self).decode_message(
-                message_type, result)
-        finally:
-            logging.getLogger().setLevel(old_level)
+        result = _DecodeCustomFieldNames(message_type, encoded_message)
+        result = super(_ProtoJsonApiTools, self).decode_message(
+            message_type, result)
         result = _ProcessUnknownEnums(result, encoded_message)
         result = _ProcessUnknownMessages(result, encoded_message)
         return _DecodeUnknownFields(result, encoded_message)