fluent-plugin-groonga 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e63680563a80a14d2293cea8cd8f6001a57ac9be
-  data.tar.gz: e443be8f4f52f2669428ae13bedfe4639aced4e4
+  metadata.gz: baa35518157dc9bbe01dd3306c3b13976b59bcb1
+  data.tar.gz: 70b51e76ca8903bf1d344319d65618f2edbd2fd3
 SHA512:
-  metadata.gz: c4d64cc5c2900ef0c17f0aefc3fa07fba7c02311bdee3bb4efe6466d234d63bedeb09514febe943a427de62e70cc810c2f632dcc135f821e14b056323197f1e7
-  data.tar.gz: b4dce7aaa76152ffddbf7c15111ff899e248ce9374525715ffd954b8e59333ab29f6049a02c81cfcf7b627ad65a3beab651474de40ff42869461df983c49d1c9
+  metadata.gz: 5ea5e5dceabf866dde7a8278cf60e5adca78e93deb8118bfa8a05c742491859499b446f570654011d8efbfbaba73e383fcbdefe3439ed84e1bd3d122fceaf27d
+  data.tar.gz: e2f4a573383da2102b7c19ffe63bd43e90d59fe0446a6bafb32bc4539e10c9b5eaa2e0a8e74c450986aa3475fb0ef686540d46c8cbc4df5b0220985a596ba5ed

data/doc/text/news.md

@@ -2,6 +2,12 @@
 
 # News
 
+## 1.2.2: 2017-11-22
+
+### Fixes
+
+* out: Fixed a bug that command execution order may be swapped.
+
 ## 1.2.1: 2017-05-01
 
 ### Fixes

data/fluent-plugin-groonga.gemspec

@@ -17,7 +17,7 @@
 
 Gem::Specification.new do |spec|
   spec.name = "fluent-plugin-groonga"
-  spec.version = "1.2.1"
+  spec.version = "1.2.2"
   spec.authors = ["Kouhei Sutou"]
   spec.email = ["kou@clear-code.com"]
   spec.summary = "Fluentd plugin to store data into Groonga and implement Groonga replication system."

data/lib/fluent/plugin/out_groonga.rb

@@ -72,11 +72,6 @@ module Fluent
         end
       end
 
-      config_section :buffer do
-        config_set_default :@type, "memory"
-        config_set_default :chunk_keys, ['tag']
-      end
-
       def configure(conf)
         compat_parameters_convert(conf, :buffer)
         super
@@ -113,7 +108,7 @@ module Fluent
         [tag, time, record].to_msgpack
       end
 
-      def formatted_to_msgpack_binary
+      def formatted_to_msgpack_binary?
         true
       end
 
@@ -633,37 +628,35 @@ module Fluent
         end
 
         def start
-          @client = nil
         end
 
         def shutdown
-          return if @client.nil?
-          @client.close
         end
 
         def execute(name, arguments={})
           command = build_command(name, arguments)
-          @client ||= Groonga::Client.new(:protocol => @protocol,
-                                          :host     => @host,
-                                          :port     => @port,
-                                          :backend  => :synchronous)
-          response = nil
-          begin
-            response = @client.execute(command)
-          rescue Groonga::Client::Error
-            @output_plugin.log.error("[output][groonga][error]",
-                                     :protocol => @protocol,
-                                     :host => @host,
-                                     :port => @port,
-                                     :command_name => name)
-            raise
-          end
-          unless response.success?
-            @output_plugin.log.error("[output][groonga][error]",
-                                     :status_code => response.status_code,
-                                     :message => response.message)
+          Groonga::Client.open(:protocol => @protocol,
+                               :host     => @host,
+                               :port     => @port,
+                               :backend  => :synchronous) do |client|
+            response = nil
+            begin
+              response = client.execute(command)
+            rescue Groonga::Client::Error
+              @output_plugin.log.error("[output][groonga][error]",
+                                       :protocol => @protocol,
+                                       :host => @host,
+                                       :port => @port,
+                                       :command_name => name)
+              raise
+            end
+            unless response.success?
+              @output_plugin.log.error("[output][groonga][error]",
+                                       :status_code => response.status_code,
+                                       :message => response.message)
+            end
+            response
           end
-          response
         end
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-groonga
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - Kouhei Sutou
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-02 00:00:00.000000000 Z
+date: 2017-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd
@@ -186,15 +186,15 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.12
+rubygems_version: 2.5.2.1
 signing_key: 
 specification_version: 4
 summary: Fluentd plugin to store data into Groonga and implement Groonga replication
   system.
 test_files:
+- test/test_input.rb
+- test/run-test.rb
 - test/test_output.rb
 - test/output/test_table_index_definition.rb
 - test/output/test_type_guesser.rb
 - test/output/test_table_definition.rb
-- test/run-test.rb
-- test/test_input.rb