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.

Sign up for GitHub

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

Jump to bottom

Removing the TODO log for computeResourceBestPossibleState function #351

Closed
alirezazamani opened this issue Jul 22, 2019 · 0 comments
Closed

Removing the TODO log for computeResourceBestPossibleState function #351

alirezazamani opened this issue Jul 22, 2019 · 0 comments

Comments

@alirezazamani
Copy link

If debugging NPE is complete, some if the logs can be removed for CRUSH based algorithm (in computeResourceBestPossibleState function).
@alirezazamani alirezazamani changed the title removing the todo log Removing the TODO log for computeResourceBestPossibleState function Jul 22, 2019
alirezazamani pushed a commit to alirezazamani/helix that referenced this issue Jul 23, 2019
Romve TODO NPE log for computeResourceBestPossibleState
d900b7e 
The logs related to NPE in computeResourceBestPossibleState is not needed anymore.

This commit fixes issue apache#351.
@alirezazamani alirezazamani mentioned this issue Jul 23, 2019
Merged
alirezazamani pushed a commit to alirezazamani/helix that referenced this issue Jul 24, 2019
Remove TODO NPE log for computeResourceBestPossibleState
4b01c4d 
The logs related to NPE in computeResourceBestPossibleState is not needed anymore.

This commit fixes issue apache#351.
alirezazamani pushed a commit to alirezazamani/helix that referenced this issue Jul 24, 2019
Remove TODO NPE log for computeResourceBestPossibleState
1dfa0ae 
The logs related to NPE in computeResourceBestPossibleState is not needed anymore.

This commit fixes issue apache#351.
junkaixue pushed a commit that referenced this issue Jul 26, 2019
@junkaixue
Remove TODO NPE log for computeResourceBestPossibleState
b429d48 
The logs related to NPE in computeResourceBestPossibleState is not needed anymore.

This commit fixes issue #351.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alirezazamani