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

Conversation

@bvellanki
Copy link
Contributor

No description provided.

---+++ Safe Mode
Safemode is useful when the admin wants to prevent Falcon users from scheduling entities in the workflow engine. This can happen when
* Hadoop clusters are being upgraded.
* Falcon cluster entity is being updated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Better to use plurality to be consistent with the previous sentence, i.e. "Falcon cluster entities are being updated".

@bvellanki
Copy link
Contributor Author

@yzheng-hortonworks Good feedback, I incorporated it into the second commit. Please review.

* Suspend/Kill of individual instances will be allowed if users want to suspend specific job operations.
* Suspend operation on schedulable entities will be allowed. This is because, a user might want to suspend entities during rolling-upgrade to handle jobs incompatible with updated versions.
* All other operations are not allowed. To enumerate,
* All entity submit, submitAndSchedule operations are not allowed.
Copy link
Contributor

Choose a reason for hiding this comment

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

redundant?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, I think it is better to spell out specifically what is allowed and what is not.

@ying-eng
Copy link
Contributor

+1 after addressing @vramachan 's comments

@vramachan
Copy link
Contributor

some minor comments.
+1

@asfgit asfgit closed this in 645e13f May 13, 2016
pallavi-rao pushed a commit to pallavi-rao/falcon that referenced this pull request Feb 16, 2018
Author: bvellanki <bvellanki@hortonworks.com>

Reviewers: "Ying Zheng <yzheng@hortonworks.com>, Venkatesan Ramachandran <vramachandran@hortonworks.com>"

Closes apache#140 from bvellanki/FALCON-1934
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