Skip to content
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

Fix #91 - Intermittent test failure on SimpleProducerConsumerTest.testSharedConsumerAckDifferentConsumer #92

Merged
merged 1 commit into from
Oct 29, 2016

Conversation

merlimat
Copy link
Contributor

Motivation

Fix intermittent test failure

Modifications

Disable consumer prefetching to ensure each consumer gets 5 messages

@yahoocla
Copy link

CLA is valid!

2 similar comments
@yahoocla
Copy link

CLA is valid!

@yahoocla
Copy link

CLA is valid!

@merlimat merlimat added this to the 1.16 milestone Oct 28, 2016
@merlimat merlimat self-assigned this Oct 28, 2016
Copy link
Contributor

@rdhabalia rdhabalia left a comment

Choose a reason for hiding this comment

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

👍

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…st.testSharedConsumerAckDifferentConsumer
@merlimat merlimat merged commit 340dc8d into apache:master Oct 29, 2016
sijie pushed a commit to sijie/pulsar that referenced this pull request Mar 4, 2018

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
* Made use of LimitsConfig in JavaInstanceConfig

* Add uncommited file
hrsakai pushed a commit to hrsakai/pulsar that referenced this pull request Dec 10, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
cbornet added a commit to cbornet/pulsar that referenced this pull request Jun 17, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…e#92)

* Parse function userConfig to Map<String, Object> (apache#15669)

Currently only {userConfig: {some-key: some-value}} is allowed which is limiting
With this change any JSON compatible object is allowed. Eg: {userConfig: {some-key: [value1, value2]}}

* Allow creating builtin functions in pulsar-admin CLI (apache#15671)

* [fix][admin] Fix check on create function class name (apache#15700)

The check was done for PYTHON or JAVA runtime but the runtime is not inferred in the admin CLI.
The fix is to do the check if a jar or py is provided instead

* [improve][function] Get function class name from the NAR when using built-in functions (apache#15693)

* Get function class name from the NAR manifest when using built-in functions

* Exclude pulsar-io.yaml from examples JAR

* Build a NAR separately from the JAR for the function examples

* Improve admin cli tes for built-in functions
cbornet added a commit to cbornet/pulsar that referenced this pull request Jun 17, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
apache#92)" (apache#93)

This reverts commit c77ed62.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants