Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Conversation

@Demogorgon314
Copy link
Member

@Demogorgon314 Demogorgon314 commented Sep 8, 2021

Currently pulsar has a hangs problem, we need fix pulsar side problem then upgrade pulsar dependency.

@Demogorgon314 Demogorgon314 self-assigned this Sep 8, 2021
@Demogorgon314 Demogorgon314 marked this pull request as draft September 18, 2021 10:17
codelipenghui pushed a commit to apache/pulsar that referenced this pull request Sep 22, 2021
Link PR in KoP:
streamnative/kop#718 and [failed test unit](https://github.com/streamnative/kop/pull/718/checks?check_run_id=3541827178#step:5:353)

### Motivation
The bug cause by #11012, the `loadOrCreatePersistentTopic` has uncaught exception may cause `getTopic` hang.

### Modifications

Caught the exception

### Verifying this change

The [unit test](#11290) added by @kaushik-develop help to isolate and reproduce the issue.

### Documentation
- [x] no-need-doc 
  
This is a bug fix, no need documentation.
@Demogorgon314
Copy link
Member Author

Pulsar 2.9.0 current don't expose getZkClient(), I need find a way to use getLocalMetadataStore() instead.

@Demogorgon314 Demogorgon314 deleted the fix-null-point-pulsar-2.9.0-SNAPSHOT branch October 20, 2021 02:25
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
Link PR in KoP:
streamnative/kop#718 and [failed test unit](https://github.com/streamnative/kop/pull/718/checks?check_run_id=3541827178#step:5:353)

### Motivation
The bug cause by apache#11012, the `loadOrCreatePersistentTopic` has uncaught exception may cause `getTopic` hang.

### Modifications

Caught the exception

### Verifying this change

The [unit test](apache#11290) added by @kaushik-develop help to isolate and reproduce the issue.

### Documentation
- [x] no-need-doc 
  
This is a bug fix, no need documentation.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant