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.

Sign up for GitHub

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

Jump to bottom

Fixed circular dependency in slf4j loggers for pulsar-storm unit tests #264

Merged
merged 1 commit into from
Mar 1, 2017
Merged

Fixed circular dependency in slf4j loggers for pulsar-storm unit tests #264

merged 1 commit into from
Mar 1, 2017

Conversation

merlimat
Copy link
Contributor

@merlimat merlimat commented Mar 1, 2017

Motivation

Change in #259 caused to pull both the log4j implementation and logger bridge :

Caused by: java.lang.IllegalStateException: Detected both 
log4j-over-slf4j.jar AND slf4j-log4j12.jar 
on the class path, preempting StackOverflowError. See also 
http://www.slf4j.org/codes.html#log4jDelegationLoop for more details.

@merlimat merlimat added this to the 1.17 milestone Mar 1, 2017
@merlimat merlimat self-assigned this Mar 1, 2017
@merlimat merlimat merged commit 0286fa8 into apache:master Mar 1, 2017
sijie added a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
@sijie
Fix pulsar-functions on locating java & python instance files (apache…
68e1f3c 
…#264)
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021
@BewareMyPower
Add tests to verify KoP with Confluent Schema Registry (apache#264)
c37d706 
This PR add a unit test to verify that when KoP is running with Confluent Schema Registry, Kafka Client with KafkaAvroSerializer or KafkaAvroDeserializer works well with different message types. The timeout is a little longer than other tests because it takes some time to start the schema registry service.
@xiaotongwang1 xiaotongwang1 mentioned this pull request Aug 4, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rdhabalia rdhabalia rdhabalia approved these changes

Assignees

@merlimat merlimat

Labels
area/test
Projects
None yet
Milestone
1.17
Development

Successfully merging this pull request may close these issues.
2 participants
@merlimat @rdhabalia