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

[INLONG-4468][Manager] Use the user-configured URL instead of the DNS of Pulsar #4479

Merged
merged 5 commits into from
Jun 2, 2022
Merged

[INLONG-4468][Manager] Use the user-configured URL instead of the DNS of Pulsar #4479

merged 5 commits into from
Jun 2, 2022

Conversation

xuesongxs
Copy link
Contributor

@xuesongxs
Copy link
Contributor Author

@dockerzhang please review.

@xuesongxs xuesongxs requested a review from dockerzhang June 2, 2022 01:41
@xuesongxs xuesongxs requested a review from healchow June 2, 2022 09:09
@healchow healchow changed the title [INLONG-4468][Manager] Use config url to create pulsar admin instead of use pulsar dns [INLONG-4468][Manager] Use the user-configured URL instead of the DNS of Pulsar to create PulsarAdmin Jun 2, 2022
@healchow healchow changed the title [INLONG-4468][Manager] Use the user-configured URL instead of the DNS of Pulsar to create PulsarAdmin [INLONG-4468][Manager] Use the user-configured URL instead of the DNS of Pulsar Jun 2, 2022
@dockerzhang dockerzhang merged commit 9183744 into apache:master Jun 2, 2022
Greedyu pushed a commit to Greedyu/inlong that referenced this pull request Jun 6, 2022
vernedeng pushed a commit to vernedeng/incubator-inlong that referenced this pull request Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improve][Manager] Use config url to create pulsar admin
4 participants