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

TAJO-1599: Implement NodeResourceManager and Status updater#577

Closed
jinossy wants to merge 17 commits intoapache:masterfrom
jinossy:TAJO-1599
Closed

TAJO-1599: Implement NodeResourceManager and Status updater#577
jinossy wants to merge 17 commits intoapache:masterfrom
jinossy:TAJO-1599

Conversation

@jinossy
Copy link
Member

@jinossy jinossy commented May 18, 2015

No description provided.

@jihoonson
Copy link
Contributor

Hi @jinossy, it's really nice work.
I've almost read your patch. I'll finish my review soon.

@jihoonson
Copy link
Contributor

+1 LGTM!

@jinossy
Copy link
Member Author

jinossy commented May 26, 2015

Thank you for the review!
I'll commit soon

@jinossy jinossy closed this May 26, 2015
eminency pushed a commit to eminency/tajo that referenced this pull request May 26, 2015
@hyunsik
Copy link
Member

hyunsik commented May 26, 2015

This patch requires updating documentation because it affects the resource configuration.

@jinossy
Copy link
Member Author

jinossy commented May 26, 2015

I was add the new key of disk configuration because if I change the configuration, it affects the current resource.
When I remove the legacy configuration, I will update the document.

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.

3 participants