Skip to content

Conversation

jsvd
Copy link
Member

@jsvd jsvd commented Sep 12, 2016

Part of the meta issue elastic/logstash#5663

this PR updates the plugin output delegation strategy to conform with elastic/logstash#5752

Using concurrency :single makes a single instance of the plugin to be shared accross all pipeline workers. However only 1 will be able to use the plugin at a time.

See #20 for a longer running rework to make the plugin be threadsafe, in order to set concurrency :shared.

@jsvd
Copy link
Member Author

jsvd commented Sep 12, 2016

With this change, I'm not sure if a major version bump is warranted. Thoughts?

@andrewvc
Copy link

LGTM

@elasticsearch-bot
Copy link

João Duarte merged this into the following branches!

Branch Commits
master b2d00ad, df50150

elasticsearch-bot pushed a commit that referenced this pull request Sep 13, 2016
@jsvd jsvd deleted the concurrency_single branch September 13, 2016 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants