-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add config file watch service #1570
Conversation
7f0939c
to
db5c785
Compare
39ed6f5
to
af53079
Compare
2342b8b
to
f293b83
Compare
af53079
to
88f603a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job, the tests of FileMonitor
are very nice !
be2-scala/src/main/scala/ch/epfl/pop/decentralized/Monitor.scala
Outdated
Show resolved
Hide resolved
be2-scala/src/test/scala/ch/epfl/pop/decentralized/MonitorSuite.scala
Outdated
Show resolved
Hide resolved
88f603a
to
454a91b
Compare
[PoP - Be2-Scala] Kudos, SonarCloud Quality Gate passed! |
[PoP - Be1-Go] Kudos, SonarCloud Quality Gate passed! |
[PoP - Fe2-Android] Kudos, SonarCloud Quality Gate passed! |
[PoP - Fe1-Web] Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pr adds the ability for the system to react on the fly to changes in the config files added in #1519.
Limitations: Without greet_server implementation we are still susceptible to having double connections (A to B and B to A, over two websockets).