[Groonga-commit] droonga/droonga-engine at 8b61c62 [master] Don't output logs from NodeMetadata module.

Back to archive index

YUKI Hiroshi null+****@clear*****
Wed Apr 8 20:24:42 JST 2015


YUKI Hiroshi	2015-04-08 20:24:42 +0900 (Wed, 08 Apr 2015)

  New Revision: 8b61c625a1578e800f80a6d353928a034532f2c9
  https://github.com/droonga/droonga-engine/commit/8b61c625a1578e800f80a6d353928a034532f2c9

  Message:
    Don't output logs from NodeMetadata module.
    
    Logs from the module breaks JSON structure of the response from serf-event-handler.

  Modified files:
    lib/droonga/node_metadata.rb

  Modified: lib/droonga/node_metadata.rb (+0 -10)
===================================================================
--- lib/droonga/node_metadata.rb    2015-04-08 20:15:09 +0900 (aa8ef7a)
+++ lib/droonga/node_metadata.rb    2015-04-08 20:24:42 +0900 (9547b48)
@@ -15,14 +15,11 @@
 
 require "json"
 
-require "droonga/loggable"
 require "droonga/path"
 require "droonga/safe_file_writer"
 
 module Droonga
   class NodeMetadata
-    include Loggable
-
     module Role
       SERVICE_PROVIDER   = "service-provider"
       ABSORB_SOURCE      = "absorb-source"
@@ -55,14 +52,12 @@ module Droonga
     end
 
     def set(key, value)
-      logger.debug("setting: #{key}=#{value}")
       key = normalize_key(key)
       @metadata[key] = value
       SafeFileWriter.write(metadata_file, JSON.pretty_generate(@metadata))
     end
 
     def delete(key)
-      logger.debug("deleting: #{key}")
       key = normalize_key(key)
       @metadata.delete(key)
       SafeFileWriter.write(metadata_file, JSON.pretty_generate(@metadata))
@@ -112,16 +107,11 @@ module Droonga
     def load
       if metadata_file.exist?
         contents = metadata_file.read
-        logger.debug("metadata loaded from file", :metadata => contents)
         unless contents.empty?
           return JSON.parse(contents, :symbolize_names => true)
         end
       end
       {}
     end
-
-    def log_tag
-      "node_metadata"
-    end
   end
 end
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Back to archive index