search

vmware-archive / haret

A strongly consistent distributed coordination system, built using proven protocols & implemented in Rust.
461 stars 18 forks source link

Change configuration file format #104

Closed jrgarcia closed 7 years ago

jrgarcia commented 7 years ago

Change configuration format to use TOML instead of JSON. This change updates the documentation, the configuration code, and the Cargo files.

Closes #76

jrgarcia commented 7 years ago

I'm pretty sure I did this correctly. I tested it and with the admin, config reports the correct configuration settings. I tested running three nodes and joining them to the same cluster. It worked just fine and the cluster status reported everything correctly. Still worth testing pretty rigorously though.

andrewjstone commented 7 years ago

Wow @jrgarcia you are my hero :) I'll give this a thorough review a bit later today.

jrgarcia commented 7 years ago

Don't get too excited until you're sure it works! 🙂

andrewjstone commented 7 years ago

LGTM. Nice work @jrgarcia. Thanks again.