You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Ajay Yadava <aj...@gmail.com> on 2014/10/19 18:05:44 UTC

Review Request 26910: FALCON-721

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

Review request for Falcon and Srikanth Sundarrajan.


Repository: falcon-git


Description
-------

Add SLA for feeds.


Diffs
-----

  client/src/main/resources/feed-0.1.xsd 3c96d80 
  common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java 34b764b 
  common/src/test/java/org/apache/falcon/entity/parser/FeedEntityParserTest.java 98cda04 
  common/src/test/resources/config/feed/feed-0.1.xml 0269fa0 
  docs/src/site/twiki/EntitySpecification.twiki b0dfb7f 

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


Testing
-------

Unit testing done for all new code.


Thanks,

Ajay Yadava

Re: Review Request 26910: FALCON-721: Add SLA for feeds

Posted by Ajay Yadava <aj...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26910/
-----------------------------------------------------------

(Updated Oct. 20, 2014, 4:21 p.m.)


Review request for Falcon and Srikanth Sundarrajan.


Summary (updated)
-----------------

FALCON-721: Add SLA for feeds


Bugs: https://issues.apache.org/jira/browse/FALCON-721
    https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/FALCON-721


Repository: falcon-git


Description (updated)
-------

Add two new properties slaLow and slaHigh for the feeds.
slaLow is intended to be used as watermark for triggering alert and slaHigh is intended to be used as watermark for finding breach of SLA.


Diffs
-----

  client/src/main/resources/feed-0.1.xsd 3c96d80 
  common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java 34b764b 
  common/src/test/java/org/apache/falcon/entity/parser/FeedEntityParserTest.java 98cda04 
  common/src/test/resources/config/feed/feed-0.1.xml 0269fa0 
  docs/src/site/twiki/EntitySpecification.twiki b0dfb7f 

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


Testing
-------

Unit testing done for all new code.


Thanks,

Ajay Yadava

Re: Review Request 26910: FALCON-721

Posted by Ajay Yadava <aj...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26910/
-----------------------------------------------------------

(Updated Oct. 19, 2014, 6:27 p.m.)


Review request for Falcon and Srikanth Sundarrajan.


Repository: falcon-git


Description
-------

Add SLA for feeds.


Diffs (updated)
-----

  client/src/main/resources/feed-0.1.xsd 3c96d80 
  common/src/main/java/org/apache/falcon/entity/parser/FeedEntityParser.java 34b764b 
  common/src/test/java/org/apache/falcon/entity/parser/FeedEntityParserTest.java 98cda04 
  common/src/test/resources/config/feed/feed-0.1.xml 0269fa0 
  docs/src/site/twiki/EntitySpecification.twiki b0dfb7f 

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


Testing
-------

Unit testing done for all new code.


Thanks,

Ajay Yadava