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

Copy MongoDB to MySQL loader into preprocessor? #7

Closed
chewisinho opened this issue Mar 1, 2016 · 4 comments
Closed

Copy MongoDB to MySQL loader into preprocessor? #7

chewisinho opened this issue Mar 1, 2016 · 4 comments
Labels

Comments

@chewisinho
Copy link
Contributor

Instead of merging in the pull request, let's consider using Alex's code as a reference and copying it (with modifications) into preprocessor.py.

Currently, preprocessor.py reads from MongoDB, writes to MongoDB (parsed), and Alex's script reads from the parsed MongoDB and writes to MySQL. It would be more streamlined if the preprocessor just reads from MongoDB and writes to MySQL directly.

Similarly, the matrix file (read from MySQL into Python matrix) should probably go in the aggregator/visualizer.

We can consider the pull request to be more like useful code that should be ported elsewhere. Anyone else have thoughts?

See #6

@alvinwan
Copy link

alvinwan commented Mar 1, 2016

Merging might make it easier to spot diffs, but either works.

The matrix might not even be need in the visualization server-side. In theory the JS on client-side could benefit from it, but I think we can leave that job to d3.js. The matrix seems nifty though, perhaps for another form of analysis down the line.

@chewisinho
Copy link
Contributor Author

It's currently on a branch, so we can just compare the branches.

As for the matrix, then, maybe we can merge it into the aggregator repo.

@alvinwan
Copy link

alvinwan commented Mar 1, 2016

Yeah, I can look at the matrix separately.

@chewisinho
Copy link
Contributor Author

This is currently complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants