dango 0.4.2 → 0.4.3

data/lib/dango/framework_base.rb

@@ -69,9 +69,9 @@ module ErrorMessage
     end
     
     str = "#{exception_class.class}:#{exception_class.message}\n#{exception_class.backtrace.pretty_inspect}"
-    if code.downcase == "u" || code.downcase == "utf8"
+    if code.to_s.downcase == "u" || code.to_s.downcase == "utf8"
       str = str.toutf8
-    elsif code.downcase == "s" || code.downcase == "sjis"
+    elsif code.to_s.downcase == "s" || code.to_s.downcase == "sjis"
       str = str.tosjis
     end
     str

data/lib/dango/server_framework.rb

@@ -108,6 +108,7 @@ class DangoServerFramework
     
     # データ送信用スレッドとQueue
     @queue_send_notice = Queue.new
+    @mutex_send_notice = Mutex.new
     Thread.start{ thread_send_notice_queue() }
     
     # サーバー開始
@@ -173,8 +174,10 @@ class DangoServerFramework
       @gserver.audit = true                 # Turn logging on.
       @gserver.debug = true                 # debug
       
-      gserver_log_fh = open(@gserver_log_file, "a")
-      @gserver.stdlog = gserver_log_fh
+      if @gserver_log_file.to_s != "" 
+        gserver_log_fh = open(@gserver_log_file, "a")
+        @gserver.stdlog = gserver_log_fh
+      end
       
       GC.disable
       open(@pid_file, "wb"){|fh| fh.write Process.pid.to_s } # pidをファイルに保存
@@ -183,7 +186,8 @@ class DangoServerFramework
       @gserver.start(@server_max_connections, 20) # backlogを20に設定
       
     rescue Exception
-      error_print("#{error_message($!, 'u')}")
+      puts error_message($!)
+#      error_print(error_message($!, 'u'))
       exit
     end
   end
@@ -253,14 +257,14 @@ class DangoServerFramework
   def starting()
     msg = "#{self.class.to_s} #{@server_host}:#{@network_port}/#{@server_max_connections} start"
     logger.warn msg
-    puts "[#{Time.now_to_s}] #{msg}"
+    puts "[#{Time.now_to_s}] #{msg}" if @gserver_log_file.to_s != "" 
   end
   
   # gserver終了処理
   def stopping()
     msg = "#{self.class.to_s} #{@server_host}:#{@network_port} stop"
     logger.warn msg
-    puts "[#{Time.now_to_s}] #{msg}"
+    puts "[#{Time.now_to_s}] #{msg}" if @gserver_log_file.to_s != "" 
   end
   
   # gserverのserveメソッド：スレッド開始処理
@@ -391,13 +395,15 @@ EOF
           end
           
           if ! session[:connected] # まだdango_connectを呼び出していないなら
-            # 接続直後のsid通知
-            send_obj = {"_sid"=>sid}
-            send_notice("_notice_sid", sid, send_obj)
-            logger.debug "_notice_sid #{sid} "
-            
-            dango_connect() # 接続時メソッド呼び出し
-            session[:connected] = true
+            @mutex_proc_thread.synchronize do
+              # 接続直後のsid通知
+              send_obj = {"_sid"=>sid}
+              send_notice("_notice_sid", sid, send_obj)
+              logger.debug "_notice_sid #{sid} "
+              
+              dango_connect() # 接続時メソッド呼び出し
+              session[:connected] = true
+            end # mutex_proc_thread
           end
         end
         
@@ -452,9 +458,9 @@ EOF
           logger.info "#{sock.inspect} #{sid} is closed"
           sock = nil
         end
-      end
+      end # mutex_proc_thread
       
-      @session_manager.close_session() # sessionの終了処理
+      @session_manager.close_session(sid) # sessionの終了処理
     end
     
   end
@@ -524,7 +530,7 @@ EOF
       "DEBUG" => Logger::DEBUG, 
     }
     log_level_hash.each do |str, const|
-      @log_level = const if str.downcase == @config['server']['log_level'].downcase
+      @log_level = const if str.downcase == @config['server']['log_level'].to_s.downcase
     end
     @log_level = @log_level || DefaultLogLevel
     
@@ -660,46 +666,49 @@ EOF
           if ! @gserver.stopped? # gserverが起動していれば
             logger.debug "heart_beat_thread_start "
             
-            # クライアントから来るハートビートのチェック
-            heart_beat_time_hash = shared[:_heart_beat_time_hash]
-            
-            # 全員分のチェック
-            socket_list.all_sid.each do |sid|
-              sock = socket_list[sid]
-              
-              if ! sock # すでにsocketが無い場合は、スキップ
-                logger.warn "!!!! heart_beat_thread_start:sock is not found. #{sid}:#{sock}"
-                next 
-              end
+            @mutex_proc_thread.synchronize do
+              # クライアントから来るハートビートのチェック
+              heart_beat_time_hash = shared[:_heart_beat_time_hash]
               
-              # クライアントから来たハートビートの時間チェックし、古ければsocketを閉じる
-              if heart_beat_time_hash[sid] && heart_beat_time_hash[sid] < Time.now - @heart_beat_receive_wait_sec
-                logger.warn "!!!! heart_beat_thread_start:heart beat old:#{sid}:#{heart_beat_time_hash[sid]}:#{sock}:#{sock.closed?}"
-#                sock.close if !sock.closed?
-                begin
-                  sock.close
-                rescue IOError
-                  logger.debug "heart_beat_thread_start:maybe already closed. IOError:#{sid}:#{sock}"
-                rescue
-                  logger.warn "heart_beat_thread_start:failed close socket:#{error_message($!, 'u')}"
+              # 全員分のチェック
+              socket_list.all_sid.each do |sid|
+                sock = socket_list[sid]
+                
+                if ! sock # すでにsocketが無い場合は、スキップ
+                  logger.warn "!!!! heart_beat_thread_start:sock is not found. #{sid}:#{sock}"
+                  next 
+                end
+                
+                # クライアントから来たハートビートの時間チェックし、古ければsocketを閉じる
+                if heart_beat_time_hash[sid] && heart_beat_time_hash[sid] < Time.now - @heart_beat_receive_wait_sec
+                  logger.warn "!!!! heart_beat_thread_start:heart beat old:#{sid}:#{heart_beat_time_hash[sid]}:#{sock}:#{sock.closed?}"
+  #                sock.close if !sock.closed?
+                  begin
+                    sock.close
+                  rescue IOError
+                    logger.debug "heart_beat_thread_start:maybe already closed. IOError:#{sid}:#{sock}"
+                  rescue
+                    logger.warn "heart_beat_thread_start:failed close socket:#{error_message($!, 'u')}"
+                  end
+                  next
                 end
               end
-            end
-          end
+            end # @mutex_proc_thread
+          end # ! @gserver.stopped?
           
-          # スレッド情報をログに出力
-          if logger.debug?
-            count += 1
-            if count % 100 == 0
-              sids = socket_list.all_sid
-              thr_list = @thr_gr_gserver_serve.list.collect{|th| th.object_id }
-              
-              logger.debug "heart_beat_thread_start:sids=#{sids.inspect}"
-              logger.debug "heart_beat_thread_start:thr_list=#{thr_list.inspect}"
-              logger.debug "heart_beat_thread_start:sids-thr_list=#{(sids-thr_list).inspect}"
-              logger.debug "heart_beat_thread_start:thr_list-sids=#{(thr_list-sids).inspect}"
-            end
-          end
+#          # スレッド情報をログに出力
+#          if logger.debug?
+#            count += 1
+#            if count % 100 == 0
+#              sids = socket_list.all_sid
+#              thr_list = @thr_gr_gserver_serve.list.collect{|th| th.object_id }
+#              
+#              logger.debug "heart_beat_thread_start:sids=#{sids.inspect}"
+#              logger.debug "heart_beat_thread_start:thr_list=#{thr_list.inspect}"
+#              logger.debug "heart_beat_thread_start:sids-thr_list=#{(sids-thr_list).inspect}"
+#              logger.debug "heart_beat_thread_start:thr_list-sids=#{(thr_list-sids).inspect}"
+#            end
+#          end
           
         rescue Exception
           logger.error "Exception heart_beat_thread_start  #{error_message($!, 'u')}"
@@ -752,7 +761,9 @@ EOF
       send_id = notice_name.to_s + Time.now.to_f.to_s + rand().to_s
       send_obj_dup["_id"] = send_id
       
-      @queue_send_notice.push([sid, send_obj_dup, encode_type].deep_dup)
+      @mutex_send_notice.synchronize do # queue_send_noticeに排他処理
+        @queue_send_notice.push([sid, send_obj_dup, encode_type].deep_dup)
+      end
     end # end sids.each
     
     self
@@ -768,13 +779,13 @@ EOF
         
         pop_data = @queue_send_notice.pop
         send_data_list.push(pop_data) # Queueデータ待ち
-#        logger.debug "thread_send_notice_queue:sending data"
-#        while(! @queue_send_notice.empty?) do       # 残りのデータもQueueに入れていく
-
-        @queue_send_notice.size.times do 
-#        while(! @queue_send_notice.empty?) do       # 残りのデータもQueueに入れていく
-          pop_data = @queue_send_notice.pop
-          send_data_list.push(pop_data) # Queueデータ待ち
+        
+        @mutex_send_notice.synchronize do # queue_send_noticeに排他処理(sizeとpopの実行の間にpushが起きないように)
+          @queue_send_notice.size.times do 
+#            while(! @queue_send_notice.empty?) do       # 残りのデータもQueueに入れていく
+            pop_data = @queue_send_notice.pop
+            send_data_list.push(pop_data) # Queueデータ待ち
+          end
         end
         
         send_data_sid_list = {}
@@ -879,29 +890,5 @@ EOF
     end
   end
   
-=begin
-  def flameloop_thread(loop_method, options={})
-    logger.warn "flameloop_thread options:#{options.inspect}"
-    options = options.deep_dup
-    options[:rate] ||= 10
-    
-    wait_sec = 1.0 / options[:rate]
-    logger.warn "flameloop_thread wait_sec:#{wait_sec.inspect}"
-    
-    th = Thread.new do 
-      loop do
-        @mutex_proc_thread.synchronize do
-          begin
-            __send__(loop_method)
-          rescue Exception
-            logger.error "flameloop_thread error:#{error_message($!, 'u')}"
-          end
-        end
-        sleep wait_sec
-      end
-    end
-  end
-=end
-  
 end
 

data/lib/dango/shared/memory_store.rb

@@ -11,8 +11,8 @@ module DangoFrameworkModule
     def initialize(config)
       @config = config
       @data = {}
-      @mutex = {}
-      @size = 0
+#      @mutex = {}
+      @mutex = Mutex.new
       
       @transact_key = {}
       @transact_thread = {}
@@ -38,11 +38,12 @@ module DangoFrameworkModule
     # 昔のトランザクション処理ロジック
     def transaction(key) 
       key = key.to_s
-      if !@mutex.has_key?(key) # 未定義キーならエラー
+      if !@data.has_key?(key) # 未定義キーならエラー
         raise(DangoFrameworkException, "Shared key is not exist.")
       end
       
-      @mutex[key].synchronize do
+#      @mutex[key].synchronize do
+      @mutex.synchronize do
 #        data = yield(@data[key].deep_dup)
         data = yield(@data[key])
         @data[key] = data.deep_dup
@@ -61,11 +62,12 @@ module DangoFrameworkModule
     # このブロックの中でcommitかrollbackか例外発生を行わなければならない
     def transaction(key) 
       key = key.to_s
-      if !@mutex.has_key?(key) # 未定義キーならエラー
+      if !@data.has_key?(key) # 未定義キーならエラー
         raise(DangoFrameworkException, "Shared key is not exist.")
       end
       
-      @mutex[key].synchronize do
+#      @mutex[key].synchronize do
+      @mutex.synchronize do
         @transact_key[Thread.current] = key # 違うスレッドからcommitできないように
         
         raise("nested transaction #{key}") if @transact_thread.has_key?(key)
@@ -141,14 +143,15 @@ module DangoFrameworkModule
     
     def [](key)
       key = key.to_s
-      if !@mutex.has_key?(key) # 未定義キーならエラー
+      if !@data.has_key?(key) # 未定義キーならエラー
         return(nil)
 #        raise(DangoFrameworkException, "Shared key is not exist.")
       end
       
 #      @mutex[key].synchronize do
+      @mutex.synchronize do
         @data[key].deep_dup
-#      end
+      end
     end
     
     def []=(key, value)
@@ -161,12 +164,13 @@ module DangoFrameworkModule
       key = key.to_s
       
       # 未定義なら定義する
-      if !@mutex.has_key?(key)
-        @mutex[key] = Mutex.new
-      end
+#      if !@data.has_key?(key)
+#        @mutex[key] = Mutex.new
+#      end
       
       # ロックしてデータを入れる
-      @mutex[key].synchronize do
+#      @mutex[key].synchronize do
+      @mutex.synchronize do
         @data[key] = value.deep_dup
       end
       

data/lib/dango/version.rb

@@ -2,7 +2,7 @@ module Dango #:nodoc:
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 4
-    TINY  = 2
+    TINY  = 3
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: dango
 version: !ruby/object:Gem::Version 
-  version: 0.4.2
+  version: 0.4.3
 platform: ruby
 authors: 
 - Keisuke Minami
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-09-19 00:00:00 +09:00
+date: 2008-09-25 00:00:00 +09:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency