You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@falcon.apache.org by aj...@apache.org on 2015/08/08 16:40:24 UTC
[08/15] falcon git commit: FALCON-1362 Colo option shouldn't be
mandatory. Contributed by Sandeep Samudrala.
FALCON-1362 Colo option shouldn't be mandatory. Contributed by Sandeep Samudrala.
Project: http://git-wip-us.apache.org/repos/asf/falcon/repo
Commit: http://git-wip-us.apache.org/repos/asf/falcon/commit/4192c19e
Tree: http://git-wip-us.apache.org/repos/asf/falcon/tree/4192c19e
Diff: http://git-wip-us.apache.org/repos/asf/falcon/diff/4192c19e
Branch: refs/heads/0.7
Commit: 4192c19e7f6fdae75abb2368ef926ece002a91a6
Parents: 40ccf44
Author: Ajay Yadava <aj...@gmail.com>
Authored: Thu Aug 6 12:07:44 2015 +0530
Committer: Ajay Yadav <aj...@inmobi.com>
Committed: Sat Aug 8 20:06:41 2015 +0530
----------------------------------------------------------------------
CHANGES.txt | 2 ++
.../org/apache/falcon/resource/AbstractEntityManager.java | 10 ++++++----
2 files changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/falcon/blob/4192c19e/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 23c9970..45a3fbb 100755
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -11,6 +11,8 @@ Trunk (Unreleased)
FALCON-796 Enable users to triage data processing issues through falcon (Ajay Yadava)
IMPROVEMENTS
+ FALCON-1362 Colo option shouldn't be mandatory(Sandeep Samudrala via Ajay Yadava)
+
FALCON-1360 Size of feed should be humanized in feed instance listing(Sndeep Samudrala via Ajay Yadava)
FALCON-1301 Improve documentation for Installation(Pragya Mittal via Ajay Yadava)
http://git-wip-us.apache.org/repos/asf/falcon/blob/4192c19e/prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java
----------------------------------------------------------------------
diff --git a/prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java b/prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java
index 478088d..43461c8 100644
--- a/prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java
+++ b/prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java
@@ -116,10 +116,12 @@ public abstract class AbstractEntityManager {
}
protected void checkColo(String colo) {
- if (!DeploymentUtil.getCurrentColo().equals(colo)) {
- throw FalconWebException.newException(
- "Current colo (" + DeploymentUtil.getCurrentColo() + ") is not " + colo,
- Response.Status.BAD_REQUEST);
+ if (StringUtils.isNotEmpty(colo) && !colo.equals("*")) {
+ if (!DeploymentUtil.getCurrentColo().equals(colo)) {
+ throw FalconWebException.newException(
+ "Current colo (" + DeploymentUtil.getCurrentColo() + ") is not " + colo,
+ Response.Status.BAD_REQUEST);
+ }
}
}