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

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

Merged
merged 1 commit into from
Sep 27, 2016
Merged

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

merged 1 commit into from
Sep 27, 2016

Conversation

merlimat
Copy link
Contributor

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

CLA is valid!

2 similar comments
@yahoocla
Copy link

CLA is valid!

@yahoocla
Copy link

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

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
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