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

[improve][misc] Sync commits from apache into 3.1_ds #297

Merged
merged 9 commits into from
Jul 30, 2024

Conversation

nikhil-ctds
Copy link
Collaborator

Motivation

This PR sync all commits from apache/branch-3.0 into 3.1_ds which are not present till 30th June.

Modifications

  • Cherry-picked commits from branch-3.0 which are not present on 3.1_ds till 30th June.

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

Check the box below or label this PR directly (if you have committer privilege).

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

heesung-sn and others added 3 commits July 29, 2024 11:23
…tive source brokers (ExtensibleLoadManagerImpl only) (apache#23064) (apache#23077)

(cherry picked from commit 55d32f2)
…apache#23050)

### Motivation

GetPartitionMetadata fail also can produce messages

- 'autoUpdatePartitionsInterval' will get partition metadata and will regularly detect partition changes

- if GetPartitionMetadata will return ServiceNotReady, client receive ServiceNotReady will close cnx

- if close the current cnx, all producers and consumers witch use this cnx will close and reconnect

(https://github.com/apache/pulsar/blob/5c6602cbb3660a696bf960f2847aac1a2ae037d2/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L323-L345)
- this will influence a lot of producers and consumers and if current time the zk not available and bundle cache not exist the topic's bundle metadata, the client can't send messages to broker because the producer lookup will fail

### Modifications

GetPartitionMetadata return MetadataError when throw MetadataStoreException

(cherry picked from commit 6fa3bcf)
(cherry picked from commit 3b3e90b)
@nikhil-ctds nikhil-ctds self-assigned this Jul 29, 2024
izumo27 and others added 6 commits July 30, 2024 09:48
(cherry picked from commit d4bbf10)
(cherry picked from commit 02f3ecc)
…ore other ops to save resources (apache#23074)

(cherry picked from commit 679a3d4)
(cherry picked from commit f08a34c)
…ISE_ACCESS_KEY (apache#23090)

(cherry picked from commit a0bdf45)
(cherry picked from commit 9897896)
…er metadata store session rebuilt (apache#23018)

(cherry picked from commit b955c65)
(cherry picked from commit 0e59678)
…nts (apache#23072)

(cherry picked from commit 77b6378)
(cherry picked from commit 487913e)
@nikhil-ctds nikhil-ctds marked this pull request as ready for review July 30, 2024 08:31
@srinath-ctds srinath-ctds merged commit f5eebae into 3.1_ds Jul 30, 2024
47 of 52 checks passed
@srinath-ctds srinath-ctds deleted the cherry-picks-into-3.1 branch July 30, 2024 09:03
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.

9 participants