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.

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

[run-tests] feat:allow client skips the non-continued deleted ranges stats #4

Closed
wants to merge 74 commits into from

Conversation

leizhiyuan
Copy link
Owner

This PR is for running tests for upstream PR apache#18263.

congbobo184 and others added 30 commits October 19, 2022 15:54
…ad position for consumer (apache#18037)

### Motivation
Fixes apache#18036

### Modifications
- The backlog eviction policy should use `asyncMarkDelete` instead of `resetCursor` in order to move the mark delete position.
…t.PersistentSubscriptionMessageDispatchStreamingDispatcherThrottlingTest#setup (apache#18115)
… tools (apache#15485)

* add a new section for websocket API

* Update site2/docs/security-tls-transport.md

Co-authored-by: Yunze Xu <[email protected]>

* Update site2/docs/security-tls-transport.md

Co-authored-by: Yunze Xu <[email protected]>

* Update site2/docs/security-tls-transport.md

Co-authored-by: Yunze Xu <[email protected]>

* updates

* updates

* updates

* update code snippet as per review comments

Co-authored-by: Yunze Xu <[email protected]>
…apache#17962)

* [feat][PIP-188] support blue-green cluster migration [part-1]

Add blue-green cluster migration

Fix dependency

* cleanup
* [improve][io] JDBC sinks: implement JDBC Batch API

* more tests and transactions support

* remove .db files

* doc

* fix batch results and thread safey

* add next flush test - fix doc - improve code readability
* [fix][doc] Point Python and C++ docs to new GitHub repos

* Fix cpp examples link

* Add notice to redirect any users
…follow BookieServiceInfo updates (apache#18133)

* [fix] PulsarRegistrationClient - implement getAllBookies and follow BookieServiceInfo updates

* remove debug
### Motivation
The current MAX_ACK_GROUP_SIZE is fixed at 1000, increase the configuration acknowledgementsGroupSize, support MAX_ACK_GROUP_SIZE configurable:
https://github.com/apache/pulsar/blob/afcdbf0e2b5fb905e1f82f0220436f8f9ec0c742/pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java#L63-L64


### Modifications
1.Add configuration acknowledgementsGroupSize;
* [feat][ci] check style for all source code

This closes apache#18131.

* fix violations

Signed-off-by: tison <[email protected]>

* no snapshot update

Signed-off-by: tison <[email protected]>

* remove mysterious dependency

Signed-off-by: tison <[email protected]>

Signed-off-by: tison <[email protected]>
… and check whether it is greater than the batch index of the previous ack (apache#18042)

Co-authored-by: leolinchen <[email protected]>
thetumbled and others added 26 commits October 25, 2022 23:16
* do not serialize field bundleStats

* add unit test.

* use org.testng.Assert.assertFalse

* add apache license header

* fix doc

* fix doc and change the package level.

Co-authored-by: fengwenzhi <[email protected]>
* add go-client chunking doc

* Update site2/docs/client-libraries-go.md

Co-authored-by: Zixuan Liu <[email protected]>

* Update site2/docs/client-libraries-go.md

Co-authored-by: Anonymitaet <[email protected]>

* Update site2/docs/client-libraries-go.md

Co-authored-by: Anonymitaet <[email protected]>

* Update site2/docs/client-libraries-go.md

Co-authored-by: Anonymitaet <[email protected]>

Co-authored-by: Zixuan Liu <[email protected]>
Co-authored-by: Anonymitaet <[email protected]>
…multiple providers (apache#18035)

* Update security-athenz.md

* improve auth overview

* address review comments

* Improve OAuth2 authentication

* improve Kerberos authentication

* Update security-athenz.md
…he#18188)

* [improve][doc] cherry-pick cpp client docs installation section

Signed-off-by: tison <[email protected]>

* Apply suggestions from code review

Co-authored-by: momo-jun <[email protected]>

* Update client-libraries-cpp.md

Signed-off-by: tison <[email protected]>
Co-authored-by: momo-jun <[email protected]>
apache#18219)

### Motivation
https://github.com/apache/pulsar/blob/b061c6ac5833c21e483368febebd0d30679a35e1/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L748-L774
The `pendingLookupRequestSemaphore` will leak when handleError. There are `LookUpRequestSemaphore` not released when removing it from `pendingRequests`

related PR: apache#17856 

### Modifications
We can't easily release the semaphore in `handleError`, because there are not only `LookUpRequest`. So release the semaphore when LookupException

### Verifying this change
Add unit test case to cover this change

### Documentation

<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->

- [ ] `doc-required` 
(Your PR needs to update docs and you will update later)

- [x] `doc-not-needed` 
bug fixs, no need doc

- [ ] `doc` 
(Your PR contains doc changes)

- [ ] `doc-complete`
(Docs have been already added)
…with active consumer leaves topic metadata inconsistent. (apache#18193)
@leizhiyuan leizhiyuan closed this Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.