Review Request 41464: FALCON-1645 : Database export

2015-12-16 Thread Venkatesan Ramachandran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41464/ --- Review request for Falcon. Repository: falcon-git Description ---

Re: Review Request 41464: FALCON-1645 : Database export

2015-12-16 Thread Balu Vellanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41464/#review110762 --- There are checkstyle violations and one minor nit. The rest of the

Re: Review Request 41464: FALCON-1645 : Database export

2015-12-16 Thread Balu Vellanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41464/#review110753 ---

[jira] [Updated] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ajay Yadava updated FALCON-1669: Assignee: baishuo > Falcon should show the stackstrace info rather than just "Unable to >

Re: Review Request 41401: Rerun API for Falcon Native Scheduler

2015-12-16 Thread Pallavi Rao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41401/#review110913 ---

[jira] [Comment Edited] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread sandeep samudrala (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061549#comment-15061549 ] sandeep samudrala edited comment on FALCON-1669 at 12/17/15 5:53 AM: -

Re: Review Request 41401: Rerun API for Falcon Native Scheduler

2015-12-16 Thread Pallavi Rao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41401/#review110914 --- Ship it! Please add the additional details (comments) that I have

[jira] [Commented] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread pavan kumar kolamuri (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061536#comment-15061536 ] pavan kumar kolamuri commented on FALCON-1669: -- +1 for the patch > Falcon should show the

Re: Review Request 41396: FALCON-1635: Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal
> On Dec. 16, 2015, 5:24 p.m., pavan kumar kolamuri wrote: > > webapp/src/test/java/org/apache/falcon/resource/AbstractSchedulerManagerJerseyIT.java, > > line 137 > > > > > > As part of falcon unit setup we should

Re: Review Request 41396: FALCON-1635: Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal
> On Dec. 17, 2015, 2:40 a.m., Ajay Yadava wrote: > > webapp/src/test/java/org/apache/falcon/resource/AbstractSchedulerManagerJerseyIT.java, > > line 134 > > > > > > I don't think fully qualified name is required.

[jira] [Commented] (FALCON-1635) Migration of EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061636#comment-15061636 ] Narayan Periwal commented on FALCON-1635: - [~ajayyadava], [~pavan kumar], Addressed the review

[jira] [Commented] (FALCON-1665) Migrate Pagination integration tests to unit tests

2015-12-16 Thread Pallavi Rao (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1665?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061586#comment-15061586 ] Pallavi Rao commented on FALCON-1665: - [~bvellanki], believe EntityManagerPaginationJerseyIT and

[jira] [Updated] (FALCON-1635) Migration of EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Narayan Periwal updated FALCON-1635: Attachment: FALCON-1635-v3.patch > Migration of EntityManagerJerseySmokeIT to use falcon

Review Request 41495: FALCON-1639 Implement update feature for native scheduler

2015-12-16 Thread Pallavi Rao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41495/ --- Review request for Falcon. Bugs: FALCON-1639

[jira] [Commented] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061330#comment-15061330 ] Ajay Yadava commented on FALCON-1669: - +1. Will commit, if there are no review comments from others.

[jira] [Commented] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread baishuo (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15061279#comment-15061279 ] baishuo commented on FALCON-1669: - can anyone give me some instruction? And I will optimize the patch :)

[jira] [Updated] (FALCON-1635) Migration of EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ajay Yadava updated FALCON-1635: Summary: Migration of EntityManagerJerseySmokeIT to use falcon unit (was: Migration of

Re: Review Request 41396: FALCON-1635: Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Ajay Yadava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41396/#review110859 ---

[jira] [Updated] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread baishuo (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] baishuo updated FALCON-1669: Attachment: FALCON-1669.patch > Falcon should show the stackstrace info rather than just "Unable to >

[jira] [Updated] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread baishuo (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] baishuo updated FALCON-1669: Attachment: (was: FALCON-1669.patch) > Falcon should show the stackstrace info rather than just

[jira] [Commented] (FALCON-1636) Add Rerun API In Falcon Native Scheduler

2015-12-16 Thread pavan kumar kolamuri (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059920#comment-15059920 ] pavan kumar kolamuri commented on FALCON-1636: -- Addressed the comments and attached updated

Re: Review Request 40779: [Falcon-1377] Add tests in falcon for the Triage API

2015-12-16 Thread Ajay Yadava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40779/#review110652 ---

Re: Review Request 41401: Rerun API for Falcon Native Scheduler

2015-12-16 Thread pavan kumar kolamuri
> On Dec. 16, 2015, 9:57 a.m., sandeep samudrala wrote: > > scheduler/src/main/java/org/apache/falcon/workflow/engine/OozieDAGEngine.java, > > line 250 > > > > > > Can details be added regarding the what entity and

[jira] [Commented] (FALCON-1635) Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059888#comment-15059888 ] Ajay Yadava commented on FALCON-1635: - It's a good idea to do only one change in one JIRA. So I will

[jira] [Updated] (FALCON-1636) Add Rerun API In Falcon Native Scheduler

2015-12-16 Thread pavan kumar kolamuri (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1636?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] pavan kumar kolamuri updated FALCON-1636: - Attachment: FALCON-1636-v2.patch > Add Rerun API In Falcon Native Scheduler >

[jira] [Commented] (FALCON-1377) Add tests in falcon for the Triage API

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059915#comment-15059915 ] Ajay Yadava commented on FALCON-1377: - [~karishmag9] The patch looks good to me, only some minor

[jira] [Commented] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread Pallavi Rao (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059849#comment-15059849 ] Pallavi Rao commented on FALCON-1669: - [~baishuo], you are not using the message in the exception.

[jira] [Commented] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread baishuo (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059868#comment-15059868 ] baishuo commented on FALCON-1669: - Had update the patch, thank you [~ajayyadava] and [~pallavi.rao] >

[jira] [Commented] (FALCON-1609) Add entity commands to falcon spring-shell CLI

2015-12-16 Thread Ajay Yadava (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15059890#comment-15059890 ] Ajay Yadava commented on FALCON-1609: - Sounds good (y) > Add entity commands to falcon spring-shell

[jira] [Updated] (FALCON-1669) Falcon should show the stackstrace info rather than just "Unable to initialize Falcon Client object"

2015-12-16 Thread baishuo (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] baishuo updated FALCON-1669: Attachment: FALCON-1669.patch > Falcon should show the stackstrace info rather than just "Unable to >

Re: Review Request 41401: Rerun API for Falcon Native Scheduler

2015-12-16 Thread pavan kumar kolamuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41401/ --- (Updated Dec. 16, 2015, 12:30 p.m.) Review request for Falcon. Changes

Re: Review Request 41396: FALCON-1635: Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41396/ --- (Updated Dec. 16, 2015, 4:39 p.m.) Review request for Falcon. Bugs:

[jira] [Commented] (FALCON-1635) Migration of EntityManagerPaginationJerseyIT and EntityManagerJerseySmokeIT to use falcon unit

2015-12-16 Thread Narayan Periwal (JIRA)
[ https://issues.apache.org/jira/browse/FALCON-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15060271#comment-15060271 ] Narayan Periwal commented on FALCON-1635: - [~ajayyadava], we are not migrating