You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/02/04 22:16:11 UTC
[GitHub] [pulsar] merlimat opened a new pull request #9486: Do not use a static map of listeners in TopicPoliciesService
merlimat opened a new pull request #9486:
URL: https://github.com/apache/pulsar/pull/9486
### Motivation
Maybe CI jobs are failing with OOM in the brokers unit tests. The Surefire worker is configured with 4 processes, each with xmx of 1G.
The problem was introduced in #7863 where a static map of listeners was added to an interface. That makes that map to contain all the `PulsarService` instances created during the tests execution and keeping references to everything else.
The map should instead be scoped to the specific instance.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #9486: Do not use a static map of listeners in TopicPoliciesService
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9486:
URL: https://github.com/apache/pulsar/pull/9486#issuecomment-773865435
@lhotari FYI
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #9486: Do not use a static map of listeners in TopicPoliciesService
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #9486:
URL: https://github.com/apache/pulsar/pull/9486#issuecomment-773865435
@lhotari FYI
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] codelipenghui merged pull request #9486: Do not use a static map of listeners in TopicPoliciesService
Posted by GitBox <gi...@apache.org>.
codelipenghui merged pull request #9486:
URL: https://github.com/apache/pulsar/pull/9486
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org