Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-13 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28839 --- Ship it! Ship It! - Rohini Palaniswamy On Nov. 13, 2013, 9:57 p.

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-13 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 13, 2013, 9:57 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-13 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28836 --- http://svn.apache.org/repos/asf/oozie/trunk/tools/src/test/java/org

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-13 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 13, 2013, 9:36 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28770 --- Ship it! Ship It! - Rohini Palaniswamy On Nov. 12, 2013, 10:41 p

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 12, 2013, 10:41 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28761 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Purshotam Shah
> On Nov. 12, 2013, 8:19 p.m., Robert Kanter wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 289 > > > > > > Instead of using a synchro

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28752 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Purshotam Shah
> On Nov. 11, 2013, 7:28 p.m., Rohini Palaniswamy wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 473 > > > > > > ?? Sharelib load att

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-12 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 12, 2013, 7:56 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-11 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28680 --- http://svn.apache.org/repos/asf/oozie/trunk/docs/src/site/twiki/DG_

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-11 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28676 --- Just few minor comments. Javadoc needs to be updated in places based

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-08 Thread Purshotam Shah
> On Nov. 8, 2013, 10:07 p.m., Rohini Palaniswamy wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 534 > > > > > > What is the type for?

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-08 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 8, 2013, 10:50 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-08 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28558 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-07 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 8, 2013, 6:33 a.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-06 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 7, 2013, 5:59 a.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-06 Thread Purshotam Shah
> On Nov. 5, 2013, 10:59 p.m., Rohini Palaniswamy wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 97 > > > > > > It does not need launch

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-06 Thread Purshotam Shah
> On Nov. 5, 2013, 6:54 p.m., Virag Kothari wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 427 > > > > > > if path is null if all dire

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-05 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28053 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-05 Thread Robert Kanter
> On Nov. 5, 2013, 6:54 p.m., Virag Kothari wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 252 > > > > > > it might happen that this p

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-05 Thread Virag Kothari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review28196 --- minor comments http://svn.apache.org/repos/asf/oozie/trunk/core/sr

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-01 Thread Purshotam Shah
> On Oct. 29, 2013, 8:11 p.m., Virag Kothari wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java, > > line 478 > > > > > > if listOfpaths is

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-11-01 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Nov. 1, 2013, 5:58 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-29 Thread Virag Kothari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review27715 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/conf/oozi

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-28 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Oct. 28, 2013, 5:58 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-21 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Oct. 21, 2013, 10:09 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-21 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- (Updated Oct. 21, 2013, 4:01 p.m.) Review request for oozie. Changes ---

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-21 Thread Purshotam Shah
> On Oct. 18, 2013, 6:30 p.m., Robert Kanter wrote: > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/service/ShareLibService.java, > > line 371 > > > > > > Please double check, but I

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-18 Thread Robert Kanter
> On Oct. 18, 2013, 6:30 p.m., Robert Kanter wrote: > > Can you also update the documentation? - Robert --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review27197

Re: Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-18 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/#review27197 --- http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/

Review Request 14741: Setup sharelib using script and pickup latest(honor ship.launcher) and remove DFS dependency at startup.

2013-10-18 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14741/ --- Review request for oozie. Bugs: OOZIE-1584 https://issues.apache.org/jira/b