Skip to content

[WIP] Visualisation Backend#406

Open
VoyTechnology wants to merge 21 commits into
masterfrom
feature/voy/visualisation-backend
Open

[WIP] Visualisation Backend#406
VoyTechnology wants to merge 21 commits into
masterfrom
feature/voy/visualisation-backend

Conversation

@VoyTechnology

Copy link
Copy Markdown
Member

Backend for the visualisation

Relates to #387 and #383

Comment thread docs/export_api_v0.1.0.md Outdated
This document defined the API used for exporting data from Raft

## Enabling Exporting ##
By default the exporting function is off, to turn it on `pfsd` must be ran with

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: "must be run"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I always make that mistake :) thanks

@VoyTechnology VoyTechnology force-pushed the feature/voy/visualisation-backend branch 2 times, most recently from e256465 to 47e910e Compare April 21, 2016 16:11
Comment thread raft/configuration.go Outdated
Uuid: nodes[i].NodeID,
CommonName: nodes[i].CommonName,
State: "unknown",
Addr: nodes[i].IP+nodes[i].Port,

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just noticed I am missing : here. Will add in next commit

Comment thread pfsd/main.go Outdated
if(*exportFlag){
raft.EnableExporting = true
exporter.NewStdServer(*exportPortFlag);
go exporter.Listen();

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be added to a wait group.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

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