airavata-courses / kr2

Fall 2018 Project Team
1 stars 1 forks source link

Peer Review Milestone 2 #63

Closed BharathaAravind closed 5 years ago

BharathaAravind commented 5 years ago

Hi Kr2 team,

Great job with the implementation. Logged into Jenkins and triggered the build jobs again. Accessed the application. Created the user and searched for job results.

1) I was curious about your flask server which is running inside your react server application which connects to zookeeper and fetches data from znodes. I didn't understand the need for it as zookeeper client is available in Javascript too. 2) The UI could be improved a bit.

Apart from these two, I found the overall architecture great and good job on implementing all the features.