logstash-output-charrington 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fbbae45d9c763ffa27448320f7150f9827732ef5b20823c078664d6deb8ab652
-  data.tar.gz: c18245e0ac6cbb11e30a824fdaf2a39dcf466e4631f307a9b1bd6aa9484bb5e1
+  metadata.gz: 95cf47c25df7b4666cfc94d7ca2c4b964164a0d76b57ce62993f8cd5c4d53bcd
+  data.tar.gz: 5a5eb5c0323f16d0b9143d6953dcf24fdf9f430e56a751412142f2f7c01dea93
 SHA512:
-  metadata.gz: d139e03b049aae067a3e5827c520eaa855e33fb889e33f51cec8cf308bb0923f04cca2b01b1b561c2578ccbe1c2ec731e6cf59918cbb101df055058403ef99c7
-  data.tar.gz: 898bb2a57d84e835a3b2e4933ebb387d1f0e030f6dc06f5c98c284aea4c51b5b6401401d85fb6bb842988d7c15497331d960735a7e57e5d9dd43c79d3f30a284
+  metadata.gz: f88562ff8b73f92e22750359178f301c5fef58bd1e2361c9d1d88e8b2c08bba7b69d001be8c5a87fbd2b741ff6bff70138d954a7f094c7794d2155c9103bea23
+  data.tar.gz: dcdd5585b34e0335693a42a04f4b0897632c7f441ad4cac419975fcca863ea6c3b8eb58154ea8f111557d2d2743c4b4b1d8659c6a5129408134961fed8bf4699

data/README.md

@@ -52,4 +52,5 @@ output {
 ```bash
 gem build logstash-output-charrington.gemspec       # build
 gem push logstash-output-charrington-x.x.x.gem      # publish
+gem owner --add <email> logstash-output-charrington # add another authorized publisher
 ```

data/lib/logstash/outputs/charrington/alter_table.rb

@@ -66,11 +66,13 @@ module Charrington
 
     def current_table_columns
       sql = "SELECT * FROM #{table_name} LIMIT 1;"
-      rs = executeQuery(sql)
+      stmt, rs = executeQuery(prep_sql(sql))
       meta_data = rs.getMetaData()
+      stmt.close unless stmt.nil?
       column_count = meta_data.getColumnCount()
-
       (1..column_count).map {|i| meta_data.getColumnName(i) }
+    ensure
+      stmt.close unless stmt.nil?
     end
 
     def execute(sql)
@@ -84,10 +86,16 @@ module Charrington
 
     def executeQuery(sql)
       stmt = connection.createStatement()
-      stmt.executeQuery(prep_sql(sql))
+      # only close the statement if something goes wrong
+      # otherwise, the caller is responsible for closing the
+      # statement when they are doen with the result set
+      return stmt, stmt.executeQuery(prep_sql(sql))
     rescue Java::OrgPostgresqlUtil::PSQLException => e
+      puts "PSQLException: #{e.message}"
+      stmt.close unless stmt.nil?
       # @logger.error("#{e.message}")
-    ensure
+    rescue => e
+      puts "Unknown exception: #{e.message}"
       stmt.close unless stmt.nil?
     end
 

data/logstash-output-charrington.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = 'logstash-output-charrington'
-  s.version = '0.2.1'
+  s.version = '0.2.2'
 
   s.licenses = ['Apache License (2.0)']
   s.summary = 'This plugin allows you to output to SQL, via JDBC'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-charrington
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - dconger