[improve] [broker] Verify at least one URL is present for cluster creation by harshithasudhakar · Pull Request #20821 · apache/pulsar

harshithasudhakar · 2023-07-17T08:46:21Z

Main/Original Issue: apache#19866

Motivation

Modifications

Checking if serviceUrl, serviceUrlTls, brokerServiceUrl, or brokerServiceUrlTls are not null. Then proceeds to throw IllegalArgumentException if either of serviceUrl, serviceUrlTls or brokerServiceUrl, brokerServiceUrlTls has not been set.

Verifying this change

Make sure that the change passes the CI checks.

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

If the box was checked, please highlight the changes

Documentation

doc
doc-required
doc-not-needed
doc-complete

Checking if `serviceUrl`, `serviceUrlTls`, `brokerServiceUrl`, or `brokerServiceUrlTls` are not null. Then proceeds to throw `IllegalArgumentException` if either of `serviceUrl`, `serviceUrlTls` or `brokerServiceUrl`, `brokerServiceUrlTls` has not been set.

github-actions · 2023-07-17T08:46:54Z

@harshithasudhakar Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

Technoboy-

Could we use StringUtils.isEmpty(getServiceUrl()) && StringUtils.isEmpty(getServiceUrlTls()) to reduce the codes ?

poorbarcode

Could you add a test for this change?

Used StringUtils.isEmpty(getServiceUrl()) && StringUtils.isEmpty(getServiceUrlTls()) to reduce the codes

harshithasudhakar · 2023-07-27T17:43:10Z

@poorbarcode I have made two testcases to make sure at least one URL is set.

Changes: 1. testUpdateClusterServiceUrl(): Tests API for updating cluster's Service URL, and checks if the retrieved serviceURL matches with it. 2. testUpdateClusterBrokerServiceUrl(): Tests API for updating cluster's Service Broker URL, and checks if the retrieved serviceBrokerURL matches with it.

harshithasudhakar · 2023-08-11T18:53:59Z

@poorbarcode @Technoboy-
I made the following changes in AdminApi2Test:

testUpdateClusterServiceUrl(): Tests API for updating cluster's Service URL, and checks if the retrieved serviceURL matches with it.
testUpdateClusterBrokerServiceUrl(): Tests API for updating cluster's Service Broker URL, and checks if the retrieved serviceBrokerURL matches with it.

harshithasudhakar · 2023-08-18T19:30:13Z

@poorbarcode @Technoboy- Hi, are there any other similar issues I can work on?

github-actions · 2023-09-18T01:47:37Z

The pr had no activity for 30 days, mark with Stale label.

harshithasudhakar · 2024-06-16T07:00:09Z

Hi, I'd like to fix the errors in this PR but I'm unable to find the log files for this. Can you help me out? How can I run the tests again?

Update ClusterDataImpl.java

404fbd2

Checking if `serviceUrl`, `serviceUrlTls`, `brokerServiceUrl`, or `brokerServiceUrlTls` are not null. Then proceeds to throw `IllegalArgumentException` if either of `serviceUrl`, `serviceUrlTls` or `brokerServiceUrl`, `brokerServiceUrlTls` has not been set.

github-actions Bot added the doc-label-missing label Jul 17, 2023

github-actions Bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Jul 17, 2023

harshithasudhakar changed the title ~~Update ClusterDataImpl.java~~ [Enhancement] Verify at least one URL is present for cluster creation Jul 17, 2023

Technoboy- reviewed Jul 24, 2023

View reviewed changes

Comment thread pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java Outdated

Technoboy- reviewed Jul 24, 2023

View reviewed changes

Comment thread pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java Outdated

Update ClusterDataImpl.java

b9c8f72

Technoboy- reviewed Jul 24, 2023

View reviewed changes

poorbarcode reviewed Jul 24, 2023

View reviewed changes

Comment thread pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java Outdated

Comment thread pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java Outdated

Update ClusterDataImpl.java

a105a47

Used StringUtils.isEmpty(getServiceUrl()) && StringUtils.isEmpty(getServiceUrlTls()) to reduce the codes

poorbarcode assigned harshithasudhakar Jul 24, 2023

poorbarcode added this to the 3.2.0 milestone Jul 24, 2023

poorbarcode added the area/config label Jul 24, 2023

Technoboy- changed the title ~~[Enhancement] Verify at least one URL is present for cluster creation~~ [improve][common] Verify at least one URL is present for cluster creation Jul 24, 2023

harshithasudhakar added 2 commits July 27, 2023 21:28

Update ClusterDataImpl.java

e6dfcf8

Update ClusterDataImplTest.java

154d259

harshithasudhakar requested review from Technoboy- and poorbarcode July 27, 2023 17:43

Update ClusterDataImplTest.java

41d5a4e

poorbarcode reviewed Jul 27, 2023

View reviewed changes

Comment thread pulsar-common/src/test/java/org/apache/pulsar/common/policies/data/ClusterDataImplTest.java Outdated

Comment thread pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterDataImpl.java Outdated

harshithasudhakar added 2 commits July 27, 2023 23:31

Update ClusterDataImpl.java

518b0d5

Update AdminApi2Test.java

b97a9d5

harshithasudhakar requested a review from poorbarcode August 12, 2023 20:38

Technoboy- modified the milestones: 3.2.0, 3.3.0 Dec 22, 2023

coderzc modified the milestones: 3.3.0, 3.4.0 May 8, 2024

lhotari modified the milestones: 4.0.0, 4.1.0 Oct 11, 2024

lhotari added the release/4.0.1 label Oct 11, 2024

lhotari added release/4.0.2 and removed release/4.0.1 labels Dec 6, 2024

lhotari added release/4.0.3 and removed release/4.0.2 labels Jan 6, 2025

lhotari added release/4.0.4 and removed release/4.0.3 labels Feb 19, 2025

lhotari added release/4.0.5 and removed release/4.0.4 labels Mar 31, 2025

lhotari added release/4.0.6 and removed release/4.0.5 labels May 21, 2025

lhotari added release/4.0.7 and removed release/4.0.6 labels Jul 30, 2025

coderzc modified the milestones: 4.1.0, 4.2.0 Sep 1, 2025

lhotari added release/4.0.8 and removed release/4.0.7 labels Sep 27, 2025

lhotari added release/4.0.9 and removed release/4.0.8 labels Nov 11, 2025

lhotari added release/4.0.10 and removed release/4.0.9 labels Feb 9, 2026

lhotari added release/4.0.11 and removed release/4.0.10 labels Apr 27, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[improve] [broker] Verify at least one URL is present for cluster creation#20821

[improve] [broker] Verify at least one URL is present for cluster creation#20821
harshithasudhakar wants to merge 15 commits intoapache:masterfrom
harshithasudhakar:patch-1

harshithasudhakar commented Jul 17, 2023 •

edited

Loading

Uh oh!

github-actions Bot commented Jul 17, 2023

Uh oh!

Uh oh!

Uh oh!

Technoboy- left a comment

Uh oh!

poorbarcode left a comment

Uh oh!

Uh oh!

Uh oh!

harshithasudhakar commented Jul 27, 2023

Uh oh!

Uh oh!

Uh oh!

harshithasudhakar commented Aug 11, 2023 •

edited

Loading

Uh oh!

harshithasudhakar commented Aug 18, 2023

Uh oh!

github-actions Bot commented Sep 18, 2023

Uh oh!

harshithasudhakar commented Jun 16, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

harshithasudhakar commented Jul 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation

Modifications

Verifying this change

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

Documentation

Uh oh!

github-actions Bot commented Jul 17, 2023

Uh oh!

Uh oh!

Uh oh!

Technoboy- left a comment

Choose a reason for hiding this comment

Uh oh!

poorbarcode left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

harshithasudhakar commented Jul 27, 2023

Uh oh!

Uh oh!

Uh oh!

harshithasudhakar commented Aug 11, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

harshithasudhakar commented Aug 18, 2023

Uh oh!

github-actions Bot commented Sep 18, 2023

Uh oh!

harshithasudhakar commented Jun 16, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

harshithasudhakar commented Jul 17, 2023 •

edited

Loading

harshithasudhakar commented Aug 11, 2023 •

edited

Loading