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

Support more config namespaces #311

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

24kpure
Copy link

@24kpure 24kpure commented Feb 8, 2023

What is the purpose of the change

With the increasingly complex business and the increasing number of microservices, the configuration of services relying on a single namespace can't meet the requirements. Maybe we should support more config namespaces, just like other configuration centers(apollonacos

Brief changelog

  • support more config namespaces
  • remove useless code in org.springframework.cloud.zookeeper.config.ZookeeperPropertySourceLocator

@pivotal-cla
Copy link

@24kpure Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@24kpure Thank you for signing the Contributor License Agreement!

@24kpure 24kpure force-pushed the moreConfigNameSpace branch 2 times, most recently from 5654d2b to fdde93e Compare February 8, 2023 07:28
@24kpure
Copy link
Author

24kpure commented Feb 10, 2023

@spencergibb Eager for your advice.

@24kpure
Copy link
Author

24kpure commented Mar 23, 2023

@msievers @ryanjbaxter help~

@24kpure
Copy link
Author

24kpure commented Apr 28, 2023

@marcingrzejszczak help~ Eager for your advice.

@24kpure
Copy link
Author

24kpure commented Jul 1, 2023

@msievers @ryanjbaxter help~

@spencergibb
Copy link
Member

The earliest this would be available is November with 4.1.0, so we'll look at it sometime between now and then.

@24kpure
Copy link
Author

24kpure commented Jul 2, 2023

The earliest this would be available is November with 4.1.0, so we'll look at it sometime between now and then.

Thanks for your response. I have raised another pr #312 about config load.Btw,I'd like to ask for your advice that local cache file is ok when zk can't not be connected.

@24kpure
Copy link
Author

24kpure commented Oct 19, 2023

The earliest this would be available is November with 4.1.0, so we'll look at it sometime between now and then.

@spencergibb please,give me some advice about this commit.

@24kpure
Copy link
Author

24kpure commented Oct 19, 2023

I have raised this pr for 8 months,but no one took part in conversation. I am sharing my idea of the project, where are members?
@spencergibb @msievers @ryanjbaxter

@24kpure
Copy link
Author

24kpure commented Dec 29, 2023

@24kpure
Copy link
Author

24kpure commented Feb 29, 2024

The earliest this would be available is November with 4.1.0, so we'll look at it sometime between now and then.

I have raised this pr for a year, Can you have a look.

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.

None yet

4 participants