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

Added replication for message store #49

Merged
merged 10 commits into from
May 9, 2014
Merged

Added replication for message store #49

merged 10 commits into from
May 9, 2014

Conversation

hawesie
Copy link
Member

@hawesie hawesie commented May 5, 2014

As discussed in Lincoln, I've added the ability for the message store to write to and query multiple databases. This supports the (external) deletion of content from the "main" message store database, but allowing queries to succeed on the replicated content. A little more information at https://github.com/hawesie/ros_datacentre/tree/hydro-devel/ros_datacentre#replication

@cburbridge
Copy link
Member

I'd be happy to merge this since if we don't use this approach for replication, as far as I see the code changes wont stop the datacentre working as it does now. @marc-hanheide I agree that it adds dependency on network communication, which might be bad. Even if we do the writes to off-robot databases in separate threads it might get uggly.

@hawesie
Copy link
Member Author

hawesie commented May 9, 2014

This now disables first attempt at replication and instead uses mongodump and mongorestore to do bulk replication. The aim is to use this to schedule replication when the robot is charging.

@Jailander
Copy link
Member

there might be a conflict with this pull request (because of me. I am sorry) I edited and merged the datacentre.launch to add machine tags this works I have tested it, but in this pull request there is a line being added to the launch file that may cause trouble.

hawesie added a commit that referenced this pull request May 9, 2014
Added replication for message store
@hawesie hawesie merged commit 570422c into strands-project:hydro-devel May 9, 2014
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.

3 participants