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

(Updated Sept. 9, 2015, 11:54 p.m.)


Review request for Falcon, Pallavi Rao, Sowmya Ramesh, and Venkat Ranganathan.


Changes
-------

This change fixes FalconWebException usage in MetadataResource as well.


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


Repository: falcon-git


Description
-------

In regression test FeedInstanceStatusTest#feedInstanceStatusRunning we are 
hitting url 
http://<falcon-node>:15000/api/instance/kill/feed/rawEmailFeed?user.name=ambari-qa
 and getting a response:

<?xml version="1.0" encoding="UTF-8"?><instancesResult>
  <status>FAILED</status>
  <message>default/org.apache.falcon.FalconWebException::null
</message>
  <requestId>default/1673513676@qtp-1725008249-101 - 
dec88465-194b-4651-a0f2-646251318f59
</requestId>
</instancesResult>


This should return proper error message.


Diffs (updated)
-----

  prism/src/main/java/org/apache/falcon/FalconWebException.java d9a3be7 
  
prism/src/main/java/org/apache/falcon/resource/metadata/AbstractMetadataResource.java
 e9c90fc 
  
prism/src/main/java/org/apache/falcon/resource/metadata/LineageMetadataResource.java
 f8b503a 
  
prism/src/main/java/org/apache/falcon/resource/metadata/MetadataDiscoveryResource.java
 23a003a 
  
prism/src/test/java/org/apache/falcon/resource/metadata/LineageMetadataResourceTest.java
 ac0e51f 
  
prism/src/test/java/org/apache/falcon/resource/metadata/MetadataDiscoveryResourceTest.java
 14f6e73 
  webapp/src/test/java/org/apache/falcon/resource/ProcessInstanceManagerIT.java 
2696e1e 

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


Testing
-------

End2End testing done. Added an IT test to catch future regressions.


Thanks,

Balu Vellanki

Reply via email to