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

Conversation

@sandeepSamudrala
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

Did you miss to attach LocalOozieCoordClientProxy class ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. I have updated the PR. Thanks @pavankumar526 for pointing it out.

}
}

@Override
Copy link
Contributor

Choose a reason for hiding this comment

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

Not Required

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes. Removed the method.

@pavankumar526
Copy link
Contributor

Looks good +1

@peeyushb
Copy link
Contributor

peeyushb commented Feb 4, 2016

+1. LGTM.

@ajayyadava
Copy link
Contributor

👍

@asfgit asfgit closed this in 49050c8 Feb 5, 2016
PraveenAdlakha pushed a commit to PraveenAdlakha/falcon that referenced this pull request Oct 3, 2016
FALCON-710 , FALCON-711, FALCON-712. Process Updates to sla changes, Start Instances and end instances. Duplicate entries.
pallavi-rao pushed a commit to pallavi-rao/falcon that referenced this pull request Feb 16, 2018
Author: sandeep <sandysmdl@gmail.com>

Reviewers: Ajay Yadava <ajayyadava@apache.org>, Pavan Kumar, Peeyush Bishnoi

Closes apache#22 from sandeepSamudrala/master
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