apache/pulsar

[Doc] controlled-failover Optimize example code

Open

#23 512 ouverte le 24 oct. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)Java (13 697 stars) (3 485 forks)batch import
doc-requiredgood first issue

Description

Search before asking

  • I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?

https://pulsar.apache.org/docs/3.0.x/client-libraries-cluster-level-failover/#controlled-failover There is unnecessary code present image image

because org.apache.pulsar.client.impl.ClientBuilderImpl#build already initialize image

What is your suggestion?

delete provider.initialize(pulsarClient); image

Any reference?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!

Guide contributeur

Stack technique
java
Domaine
documentation
Type d'issue
documentation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
The issue is to remove the unnecessary `provider.initialize(pulsarClient);` call from the example code in the controlled failover documentation. The example is on the page https://pulsar.apache.org/docs/3.0.x/client libraries cluster level failover/#controlled failover. The fix is a single line deletion. No additional research needed.