search

logstash-plugins / logstash-filter-jdbc_streaming

A Logstash filter that can enrich events with data from a database
Apache License 2.0
12 stars 23 forks source link

Fix driver loading #35

Closed robbavey closed 5 years ago

robbavey commented 5 years ago

Making the same fix to driver loading that was applied to the JDBC input in https://github.com/logstash-plugins/logstash-input-jdbc/pull/356

This commit also reworks the test to force driver loading in the integration tests by removing the postgres gem as a dependency and doing all the database setup work in an external sql script)

This fixes #34 (the fix previously applied in #31 did not work correctly)

andsel commented 5 years ago

LGTM

elasticsearch-bot commented 5 years ago

Robert Bavey merged this into the following branches!

Branch Commits
master 21ebb04c5dc4b1637c53a6b8ae2b602494741a35, 0e22d310760ff80e927932b7f7535577a608999e