You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Ruslan Ostafiychuk <ro...@hortonworks.com> on 2014/11/05 19:03:32 UTC

Review Request 27629: Rest end-point entities/summary has changed

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

Review request for Falcon.


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


Repository: falcon-git


Description
-------

The rest endpoint which used to look like:
http://node1.internal:15000/api/entities/summary/feed/corp-b46b6bfa
has changed to:
http://node1.internal:15000/api/entities/summary/feed?cluster=corp-b46b6bfa


Diffs
-----

  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/interfaces/IEntityManagerHelper.java e1d808e 

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


Testing
-------


Thanks,

Ruslan Ostafiychuk


Re: Review Request 27629: Rest end-point entities/summary has changed

Posted by Raghav Gautam <ra...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27629/#review59995
-----------------------------------------------------------

Ship it!


LGTM. Please make sure that tests are passing.

- Raghav Gautam


On Nov. 5, 2014, 10:03 a.m., Ruslan Ostafiychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27629/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 10:03 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-865
>     https://issues.apache.org/jira/browse/FALCON-865
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> The rest endpoint which used to look like:
> http://node1.internal:15000/api/entities/summary/feed/corp-b46b6bfa
> has changed to:
> http://node1.internal:15000/api/entities/summary/feed?cluster=corp-b46b6bfa
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/interfaces/IEntityManagerHelper.java e1d808e 
> 
> Diff: https://reviews.apache.org/r/27629/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ruslan Ostafiychuk
> 
>


Re: Review Request 27629: Rest end-point entities/summary has changed

Posted by Ruslan Ostafiychuk <ro...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27629/
-----------------------------------------------------------

(Updated Nov. 6, 2014, 1:21 p.m.)


Review request for Falcon.


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


Repository: falcon-git


Description
-------

The rest endpoint which used to look like:
http://node1.internal:15000/api/entities/summary/feed/corp-b46b6bfa
has changed to:
http://node1.internal:15000/api/entities/summary/feed?cluster=corp-b46b6bfa


Diffs
-----

  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/interfaces/IEntityManagerHelper.java e1d808e 

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


Testing (updated)
-------

Tested. Some tests fail because of app issue.


Thanks,

Ruslan Ostafiychuk