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

Dledger properties configuration file Support #171

Closed
mxsm opened this issue Jul 2, 2022 · 5 comments
Closed

Dledger properties configuration file Support #171

mxsm opened this issue Jul 2, 2022 · 5 comments

Comments

@mxsm
Copy link
Contributor

mxsm commented Jul 2, 2022

Deleger configuration using the encoding, should support configuration file configuration。The DLedgerConfig config class should can config in properties config file.

@mxsm
Copy link
Contributor Author

mxsm commented Jul 2, 2022

for example, dataStorePath we can config in properties file.

dataStorePath=/root/path/mxsm

@mxsm mxsm changed the title Dleder can Support configuration file? Dledger can Support configuration file? Jul 15, 2022
@mxsm mxsm changed the title Dledger can Support configuration file? Dledger properties configuration file Support Jul 16, 2022
@mxsm
Copy link
Contributor Author

mxsm commented Jul 16, 2022

sample

dledger.properties

group=dleger
peers=n0-localhost:20911

how to use

java -jar target/DLedger.jar server -c /tmp/config/dledger.properties

@TheR1sing3un
Copy link
Contributor

Hi~ Maybe we have done the same work.
In this pr, it supports configuration(yaml).
https://github.com/openmessaging/dledger/pull/150/commits
And I build it basing Multi-Raft structure.
#82
Could we talk about this? 😂

@mxsm
Copy link
Contributor Author

mxsm commented Jul 18, 2022

Hi~ Maybe we have done the same work. In this pr, it supports configuration(yaml). https://github.com/openmessaging/dledger/pull/150/commits And I build it basing Multi-Raft structure. #82 Could we talk about this? 😂

I watched the RP, when you pr be merged,I'll build on this for other feature development.

@TheR1sing3un
Copy link
Contributor

Hi~ Maybe we have done the same work. In this pr, it supports configuration(yaml). https://github.com/openmessaging/dledger/pull/150/commits And I build it basing Multi-Raft structure. #82 Could we talk about this? 😂

I watched the RP, when you pr be merged,I'll build on this for other feature development.

ok~looks great

@mxsm mxsm closed this as completed Jul 22, 2022
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

No branches or pull requests

2 participants