YUKI Hiroshi
null+****@clear*****
Thu Oct 31 15:51:43 JST 2013
YUKI Hiroshi 2013-10-31 15:51:43 +0900 (Thu, 31 Oct 2013) New Revision: 0c270a4d01261d3ae90b50eb315466241ecef5ee https://github.com/droonga/fluent-plugin-droonga/commit/0c270a4d01261d3ae90b50eb315466241ecef5ee Message: Execute Rroonga's select outside of "if" statements Modified files: lib/droonga/watcher.rb Modified: lib/droonga/watcher.rb (+12 -9) =================================================================== --- lib/droonga/watcher.rb 2013-10-31 15:51:21 +0900 (3d62e22) +++ lib/droonga/watcher.rb 2013-10-31 15:51:43 +0900 (3425acb) @@ -200,9 +200,10 @@ module Droonga def sweep_orphan_queries(queries) queries.each do |query| - if @subscriber_table.select do |subscriber| - subscriber.subscriptions =~ query - end.empty? + related_subscribers = @subscriber_table.select do |subscriber| + subscriber.subscriptions =~ query + end + if related_subscribers.empty? delete_query(query) end end @@ -210,18 +211,20 @@ module Droonga def sweep_orphan_keywords(keywords) keywords.each do |keyword| - if @query_table.select do |query| - query.keywords =~ keyword - end.empty? + related_queries = @query_table.select do |query| + query.keywords =~ keyword + end + if related_queries.empty? keyword.delete end end end def sweep_orphan_route(route) - if @subscriber_table.select do |subscriber| - subscriber.route == route - end.empty? + related_subscribers = @subscriber_table.select do |subscriber| + subscriber.route == route + end + if related_subscribers.empty? route.delete end end -------------- next part -------------- HTML����������������������������...Download