Skip to content

Added log4j with info logs for pulsar-broker unit tests #37

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
Merged
merlimat merged 1 commit into from
Sep 27, 2016
Merged

Added log4j with info logs for pulsar-broker unit tests #37

merlimat merged 1 commit into from
Sep 27, 2016

Conversation

@merlimat
Copy link
Contributor

@merlimat merlimat commented Sep 23, 2016

Motivation

Add default log4j config for pulsar-broker unit tests

@merlimat merlimat added this to the 1.15 milestone Sep 23, 2016
@merlimat merlimat self-assigned this Sep 23, 2016
@yahoocla
Copy link

yahoocla commented Sep 23, 2016

CLA is valid!

2 similar comments
@yahoocla
Copy link

yahoocla commented Sep 23, 2016

CLA is valid!

@yahoocla
Copy link

yahoocla commented Sep 23, 2016

CLA is valid!

rdhabalia
rdhabalia approved these changes Sep 24, 2016
View reviewed changes
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.

👍

@merlimat
Copy link
Contributor Author

merlimat commented Sep 26, 2016

Apparently, Travis has a limit of 4MB for job output and with this change that would be exceeded. I'll add the log4j conf file with default to off for now.

@merlimat merlimat merged commit 728091e into apache:master Sep 27, 2016
sijie pushed a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
@srkukarni @sijie
Asyncack (apache#37)
7fbe342 
* Create pulsar-functions module (#1)

* Create pulsar-functions module

* rename `sdk` package to `api`

* Added the first cut of the Java interface for Pulsar functions (#2)

* Made ack async
massakam pushed a commit to massakam/pulsar that referenced this pull request Aug 6, 2020
@hrsakai
Merge pull request apache#37 from freeart/master
1869d2f 
added serialize/deserialize methods to set custom startMessageId in createReader
hrsakai pushed a commit to hrsakai/pulsar that referenced this pull request Dec 10, 2020
@wolfstudy
Merge pull request apache#37 from wolfstudy/xiaolong/fix-zlib-compress
987d2f0 
[issue:34] Fix zlib compress Decompress logic
@xiaotongwang1 xiaotongwang1 mentioned this pull request Aug 4, 2021
Closed
@ZHr-UChiHa ZHr-UChiHa mentioned this pull request Jun 3, 2023
Open
2 tasks
nodece referenced this pull request in nodece/pulsar Mar 12, 2025
@nodece
[feat][broker] Add metrics for marker and snapshot (#37)
63dd4e9 
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
nodece referenced this pull request in nodece/pulsar Mar 27, 2025
@nodece
[feat][broker] Add metrics for marker and snapshot (#37)
ea99d79 
Signed-off-by: Zixuan Liu <nodeces@gmail.com>
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.15

Development

Successfully merging this pull request may close these issues.

3 participants

@merlimat @yahoocla @rdhabalia