Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread Suman Karumuri
On May 13, 2014, 12:26 a.m., David McLaughlin wrote: File Attachment: job summary of cron job - Screen Shot 2014-05-11 at 1.37.54 PM.png https://reviews.apache.org/r/21250/#fcomment18 There is no distinction between label/value nor whitespace or borders to separate different

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/#review43160 ---

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/#review43164 --- Ship it! Ship It! - David McLaughlin On May 15, 2014, 9:07

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/#review43156 --- Ship it! Display and java code LGTM. I'll let David field the JS

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/ --- (Updated May 15, 2014, 9:07 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-16 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/ --- (Updated May 15, 2014, 12:46 a.m.) Review request for Aurora, David McLaughlin

Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-12 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/ --- Review request for Aurora, David McLaughlin and Bill Farner. Bugs: AURORA-384

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-12 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/#review42789 --- File Attachment: job summary of cron job - Screen Shot 2014-05-11

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-12 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21250/ --- (Updated May 13, 2014, 1:12 a.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21250: Removed cron jobs table from role and env page. Added cron job summary to job page.

2014-05-12 Thread Suman Karumuri
On May 13, 2014, 12:26 a.m., David McLaughlin wrote: File Attachment: job summary of cron job - Screen Shot 2014-05-11 at 1.37.54 PM.png https://reviews.apache.org/r/21250/#fcomment14 There is no distinction between label/value nor whitespace or borders to separate different