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

Logs not printed for unit tests #244

Closed
jai1 opened this issue Feb 24, 2017 · 1 comment
Closed

Logs not printed for unit tests #244

jai1 opened this issue Feb 24, 2017 · 1 comment

Comments

@jai1
Copy link
Contributor

jai1 commented Feb 24, 2017

Expected behavior

Log messages (written in test cases) should be printed on the console during build.

Actual behavior

Only log messages generated by JUnit like STARTING TEST, Test succeeded, Before configuration is be printed, not the ones written in test case.

Steps to reproduce

mvn test -B

System configuration

Pulsar version: x.y

@jai1 jai1 added the area/test label Feb 24, 2017
@jai1 jai1 added this to the 1.18 milestone Feb 24, 2017
@merlimat
Copy link
Contributor

Actually the problem is that you enable logs, you'll reach the max log size for the Travis build and that will make the build fail

@jai1 jai1 closed this as completed Feb 27, 2017
@jai1 jai1 added the wontfix label Feb 27, 2017
@merlimat merlimat removed this from the 1.18 milestone Mar 11, 2017
hangc0276 pushed a commit to hangc0276/pulsar that referenced this issue May 26, 2021
Master issue: apache#241

This PR add support for Kafka CreateTopics request so that topics can be created by Kafka's AdminClient. Related tests are added to verify

The validateOnly field introduced from CreateTopics request v1 is not processed. The validateOnly field only checks the topics can be created as specified but not creates anything. It's not a important feature currently so leave it as a future task.


* Add handler for CreateTopics request

* Fix wrong numTopics and default namespace

* Add test for CreateTopics request handler

* Handle the corner case when topic name is invalid

* Add this to make code more readable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants