New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Esper listener can be disabled! #6

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@Fabryprog
Contributor

Fabryprog commented Sep 15, 2016

Some esper statments are listener free (create window, create table).
With this fix esper component not create the listener (using listen attribute = false)

@Fabryprog

This comment has been minimized.

Show comment
Hide comment
@Fabryprog

Fabryprog Oct 4, 2016

Contributor

+1

Contributor

Fabryprog commented Oct 4, 2016

+1

@Fabryprog

This comment has been minimized.

Show comment
Hide comment
@Fabryprog

Fabryprog Oct 25, 2016

Contributor

+1, there are any news?

Contributor

Fabryprog commented Oct 25, 2016

+1, there are any news?

@ullgren

This comment has been minimized.

Show comment
Hide comment
@ullgren

ullgren Oct 28, 2016

Contributor

Could you please also add a unit test that tests and verifies when the listen is set to false.

Thanks for your contribution.

Contributor

ullgren commented Oct 28, 2016

Could you please also add a unit test that tests and verifies when the listen is set to false.

Thanks for your contribution.

@ullgren

This comment has been minimized.

Show comment
Hide comment
@ullgren

ullgren Oct 28, 2016

Contributor
Contributor

ullgren commented Oct 28, 2016

@Fabryprog

This comment has been minimized.

Show comment
Hide comment
@Fabryprog

Fabryprog Oct 28, 2016

Contributor

I will done so as possible. Thanks!

P.S. https://camel-extra.atlassian.net/projects/CAMEX/issues/CAMEX-73

Contributor

Fabryprog commented Oct 28, 2016

I will done so as possible. Thanks!

P.S. https://camel-extra.atlassian.net/projects/CAMEX/issues/CAMEX-73

@ullgren ullgren self-assigned this Oct 28, 2016

@Fabryprog

This comment has been minimized.

Show comment
Hide comment
@Fabryprog

Fabryprog Oct 28, 2016

Contributor

test done!

Contributor

Fabryprog commented Oct 28, 2016

test done!

@ullgren

This comment has been minimized.

Show comment
Hide comment
@ullgren

ullgren Oct 28, 2016

Contributor

Merged into master. Thanks for your contribution.

Contributor

ullgren commented Oct 28, 2016

Merged into master. Thanks for your contribution.

@ullgren ullgren closed this Oct 28, 2016

@Fabryprog

This comment has been minimized.

Show comment
Hide comment
@Fabryprog

Fabryprog Oct 28, 2016

Contributor

Thanks!

Contributor

Fabryprog commented Oct 28, 2016

Thanks!

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