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

[fix] [client] Fix memory leak when publishing encountered a corner case error #23738

Merged
merged 20 commits into from
Dec 20, 2024
Merged

[fix] [client] Fix memory leak when publishing encountered a corner case error #23738

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
fc1c70f
[fix] [client] Fix memory leak and publish stuck when publishing
poorbarcode Dec 16, 2024
993ef1b
checkstyle
poorbarcode Dec 17, 2024
3497e0c
fix bug
poorbarcode Dec 17, 2024
65eabda
fix tests
poorbarcode Dec 17, 2024
f69c975
fix tests
poorbarcode Dec 17, 2024
fe6b38e
address comments
poorbarcode Dec 18, 2024
aca479c
Update pulsar-client/src/main/java/org/apache/pulsar/client/impl/Prod…
poorbarcode Dec 18, 2024
90c6769
address comments
poorbarcode Dec 18, 2024
6fe072c
address comments
poorbarcode Dec 18, 2024
9904bdd
address comments
poorbarcode Dec 18, 2024
4d47f75
checkstyle
poorbarcode Dec 18, 2024
6503877
address comments
poorbarcode Dec 18, 2024
a872ce6
improve the test
poorbarcode Dec 19, 2024
4361125
improve the test
poorbarcode Dec 19, 2024
d8cf79b
address comment
poorbarcode Dec 19, 2024
aad0179
address comment
poorbarcode Dec 19, 2024
918c648
address comment
poorbarcode Dec 19, 2024
ea6a2d7
Update pulsar-client/src/main/java/org/apache/pulsar/client/impl/Type…
BewareMyPower Dec 19, 2024
f525ebf
Removed meaningless override of doInitConf
BewareMyPower Dec 19, 2024
9f99fd2
address comments
poorbarcode Dec 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed meaningless override of doInitConf
  • Loading branch information
@BewareMyPower
BewareMyPower authored Dec 19, 2024
commit f525ebf9fa0ac6ed79cf0d42caaa72d7766e183e
5 changes: 0 additions & 5 deletions pulsar-broker/src/test/java/org/apache/pulsar/client/impl/ProducerMemoryLeakTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,6 @@ protected void cleanup() throws Exception {
super.internalCleanup();
}

@Override
protected void doInitConf() throws Exception {
super.doInitConf();
}

@Test
public void testSendQueueIsFull() throws Exception {
final String topicName = BrokerTestUtil.newUniqueName("persistent://public/default/tp_");
Expand Down
Loading