Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

Conversation

@yeongeon
Copy link

Add basic monitoring page

@jinossy
Copy link
Member

jinossy commented Jul 31, 2015

@hyunsik
nv.d3.js license seems to apache 2.0. Should we add d3 license to LICENSE file ?
https://github.com/novus/nvd3/blob/master/LICENSE.md

Copy link
Contributor

Choose a reason for hiding this comment

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

Simply Monitoring looks more appropriate because we don't provide any complex or advanced monitoring.

@hyunsik
Copy link
Member

hyunsik commented Aug 10, 2015

@jinossy We don't need to add d3 license to LICENSE file if it is under ASF 2.0.

@hyunsik
Copy link
Member

hyunsik commented Aug 10, 2015

For clear, it would be better if you add this license (https://github.com/novus/nvd3/blob/master/LICENSE.md) into LICENSE file.

@hyunsik
Copy link
Member

hyunsik commented Aug 16, 2015

Hi @yeongeon,

This work looks great to me. But, it includes per-node monitoring feature. But, it is not fit in Tajo web page because Tajo basically aims at a number of nodes.

If you want to put a single page monitoring to Tajo, it would be great if the page is a summary page to give an overview of cluster, query, and scheduling information.

So, this work needs more work to remove worker parts and add some more information for an overview of them. For this, it would be great if you continue this work in a separate branch because 0.11.0 release is approaching.

I'll look forward to your comments. Thanks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants