You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Praveen Adlakha <ad...@gmail.com> on 2016/01/28 08:04:55 UTC

Review Request 42855: Error message when we hit falcon server in place of prism

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

Review request for Falcon.


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


Repository: falcon-git


Description
-------

Error message when we hit falcon server in place of prism


Diffs
-----

  webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 

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


Testing
-------

$ bin/falcon entity -type process -name process1 -delete
Delete on server is not supported.Please run your operation on Prism.


Thanks,

Praveen Adlakha


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Deepak Barr <de...@gmail.com>.

> On Jan. 28, 2016, 7:26 a.m., Deepak Barr wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Not sure but I think its better to handle this in SchedulableEntityManagerProxy class instead.
> 
> Deepak Barr wrote:
>     It will save the network call i think.
> 
> pavan kumar kolamuri wrote:
>     Deepak If i understand you correct,  SchedulableEntityManagerProxy is only for Prism, Server wont contain this class.

Ignored these comments :)


- Deepak


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by pavan kumar kolamuri <pa...@gmail.com>.

> On Jan. 28, 2016, 7:26 a.m., Deepak Barr wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Not sure but I think its better to handle this in SchedulableEntityManagerProxy class instead.
> 
> Deepak Barr wrote:
>     It will save the network call i think.

Deepak If i understand you correct,  SchedulableEntityManagerProxy is only for Prism, Server wont contain this class.


- pavan kumar


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Ajay Yadava <aj...@gmail.com>.

> On Jan. 28, 2016, 7:26 a.m., Deepak Barr wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Not sure but I think its better to handle this in SchedulableEntityManagerProxy class instead.
> 
> Deepak Barr wrote:
>     It will save the network call i think.
> 
> pavan kumar kolamuri wrote:
>     Deepak If i understand you correct,  SchedulableEntityManagerProxy is only for Prism, Server wont contain this class.
> 
> Deepak Barr wrote:
>     Ignored these comments :)

Best place to handle them is server class.


- Ajay


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Deepak Barr <de...@gmail.com>.

> On Jan. 28, 2016, 7:26 a.m., Deepak Barr wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Not sure but I think its better to handle this in SchedulableEntityManagerProxy class instead.

It will save the network call i think.


- Deepak


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Deepak Barr <de...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42855/#review116734
-----------------------------------------------------------




webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java (line 74)
<https://reviews.apache.org/r/42855/#comment177799>

    Not sure but I think its better to handle this in SchedulableEntityManagerProxy class instead.


- Deepak Barr


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Praveen Adlakha <ad...@gmail.com>.

> On Jan. 28, 2016, 8:17 a.m., pavan kumar kolamuri wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Even the same we should do for submit and other API's which can be done only through prism.

Right now I am also aware submit,update and delete are not supported by falcon server.Can you confirm I have to make changes for only these three.


- Praveen


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by pavan kumar kolamuri <pa...@gmail.com>.

> On Jan. 28, 2016, 8:17 a.m., pavan kumar kolamuri wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Even the same we should do for submit and other API's which can be done only through prism.
> 
> Praveen Adlakha wrote:
>     Right now I am also aware submit,update and delete are not supported by falcon server.Can you confirm I have to make changes for only these three.

Yes correct, we can think of better way to solve this. Give me some time


- pavan kumar


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Ajay Yadava <aj...@gmail.com>.

> On Jan. 28, 2016, 8:17 a.m., pavan kumar kolamuri wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     Even the same we should do for submit and other API's which can be done only through prism.
> 
> Praveen Adlakha wrote:
>     Right now I am also aware submit,update and delete are not supported by falcon server.Can you confirm I have to make changes for only these three.
> 
> pavan kumar kolamuri wrote:
>     Yes correct, we can think of better way to solve this. Give me some time

submit, submitAndSchedule, update and delete are the apis that need to be implemented like this.


- Ajay


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by pavan kumar kolamuri <pa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42855/#review116739
-----------------------------------------------------------




webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java (line 74)
<https://reviews.apache.org/r/42855/#comment177803>

    Even the same we should do for submit and other API's which can be done only through prism.


- pavan kumar kolamuri


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Deepak Barr <de...@gmail.com>.

> On Jan. 28, 2016, 12:01 p.m., Praveen Adlakha wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     cool...One more command that works on prism is submitAndSchedule will add for that too..Will wait for your response before making any changes also please let me know why this approach is bad.
> 
> Ajay Yadava wrote:
>     I actually like this approach, Only thing I would like to be improved is to send Forbidden (HTTP Status code-403)

403 is more about access and permission I feel. We can think of HTTP 501 or 405 instead.


- Deepak


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Ajay Yadava <aj...@gmail.com>.

> On Jan. 28, 2016, 12:01 p.m., Praveen Adlakha wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     cool...One more command that works on prism is submitAndSchedule will add for that too..Will wait for your response before making any changes also please let me know why this approach is bad.

I actually like this approach, Only thing I would like to be improved is to send Forbidden (HTTP Status code-403)


- Ajay


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Ajay Yadava <aj...@gmail.com>.

> On Jan. 28, 2016, 12:01 p.m., Praveen Adlakha wrote:
> > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java, line 74
> > <https://reviews.apache.org/r/42855/diff/1/?file=1224655#file1224655line74>
> >
> >     cool...One more command that works on prism is submitAndSchedule will add for that too..Will wait for your response before making any changes also please let me know why this approach is bad.
> 
> Ajay Yadava wrote:
>     I actually like this approach, Only thing I would like to be improved is to send Forbidden (HTTP Status code-403)
> 
> Deepak Barr wrote:
>     403 is more about access and permission I feel. We can think of HTTP 501 or 405 instead.

It is exactly about permission. You are not allowed to call delete on server, but prism is allowed to call it. Since the method is implemented but you are not allowed to use it, you shouldn't return 501. Similarly the common use case for 405 is when a particular method is not allowed on that resource e.g. GET instead of POST, which is not the case either. So 403 seems better to me.


- Ajay


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


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by Praveen Adlakha <ad...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42855/#review116759
-----------------------------------------------------------




webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java (line 74)
<https://reviews.apache.org/r/42855/#comment177839>

    cool...One more command that works on prism is submitAndSchedule will add for that too..Will wait for your response before making any changes also please let me know why this approach is bad.



webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java (line 79)
<https://reviews.apache.org/r/42855/#comment177838>

    I believe this is a better error msg as we are telling the user what he is doing wrong and what he should do next.


- Praveen Adlakha


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>


Re: Review Request 42855: Error message when we hit falcon server in place of prism

Posted by sandeep samudrala <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42855/#review116730
-----------------------------------------------------------




webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java (line 79)
<https://reviews.apache.org/r/42855/#comment177797>

    Delete api is not supported on server?


- sandeep samudrala


On Jan. 28, 2016, 7:04 a.m., Praveen Adlakha wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42855/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2016, 7:04 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1774
>     https://issues.apache.org/jira/browse/FALCON-1774
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Error message when we hit falcon server in place of prism
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java afa2b6f 
> 
> Diff: https://reviews.apache.org/r/42855/diff/
> 
> 
> Testing
> -------
> 
> $ bin/falcon entity -type process -name process1 -delete
> Delete on server is not supported.Please run your operation on Prism.
> 
> 
> Thanks,
> 
> Praveen Adlakha
> 
>