Skip to content

Allow Emitter to use a custom ExecutorService (close #278)#259

Merged
Miranda Wilson (mscwilson) merged 1 commit intosnowplow:release/0.11.0from
AcidFlow:feature/custom-emitter-executor
Nov 30, 2021
Merged

Allow Emitter to use a custom ExecutorService (close #278)#259
Miranda Wilson (mscwilson) merged 1 commit intosnowplow:release/0.11.0from
AcidFlow:feature/custom-emitter-executor

Conversation

@AcidFlow
Copy link
Copy Markdown
Contributor

This commit allows a user to use its own ExecutorService in the Emitter
to have a more granular control over threads created by an Emitter.

This commit allows a user to use its own ExecutorService in the Emitter
to have a more granular control over threads created by an Emitter.
@snowplowcla Snowplow CLA bot (snowplowcla) added the cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed. label Jun 29, 2020
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thank you for the contribution, we'll release this with 0.11.0.

@svzdvd
Copy link
Copy Markdown

any plan for the 0.11.0 release?

@paulboocock
Copy link
Copy Markdown
Contributor

Hi Davide Savazzi (@svzdvd)
It's expected in Q2 2021, so work should begin on this in the next few weeks.

@mscwilson Miranda Wilson (mscwilson) changed the base branch from master to release/0.11.0 November 30, 2021 11:17
@mscwilson Miranda Wilson (mscwilson) changed the title Allow Emitter to use a custom ExecutorService Allow Emitter to use a custom ExecutorService (close #278) Nov 30, 2021
@mscwilson
Copy link
Copy Markdown
Contributor

Thanks again for your contribution. It's now been merged in for release 0.11.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed.

Development

Successfully merging this pull request may close these issues.

5 participants