falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Venkatesan Ramachandran <me.venk...@gmail.com>
Subject Re: Review Request 41916: FALCON-1584 : Falcon allows invalid hadoop queue name for schedulable feed entities
Date Wed, 10 Feb 2016 22:32:21 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41916/
-----------------------------------------------------------

(Updated Feb. 10, 2016, 10:32 p.m.)


Review request for Falcon.


Changes
-------

+ Feed entity - check replication and retention queues
+ Support for Process Entity
+ Review board comments


Repository: falcon-git


Description
-------

Validate the hadoop cluster queue name specified in the Feed entity during feed submit.

The implementation uses Resource Manager REST API to get hadoop cluster queue names.


Diffs (updated)
-----

  common/src/main/java/org/apache/falcon/entity/FeedHelper.java b3aaaab 
  common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java 981e730 
  common/src/main/java/org/apache/falcon/entity/parser/ProcessEntityParser.java 10a5265 
  common/src/main/java/org/apache/falcon/util/HadoopQueueUtil.java PRE-CREATION 
  common/src/test/java/org/apache/falcon/entity/FeedHelperTest.java 95d10c4 
  common/src/test/java/org/apache/falcon/util/HadoopQueueUtilTest.java PRE-CREATION 
  common/src/test/resources/config/feed/feed-schedulerinfo-1.json PRE-CREATION 
  common/src/test/resources/config/feed/feed-schedulerinfo-2.json PRE-CREATION 

Diff: https://reviews.apache.org/r/41916/diff/


Testing
-------

Unit tests
Manual end to end tests
Secure cluster test


Thanks,

Venkatesan Ramachandran


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message