Re: Review Request 26836: Cron documentation

2014-10-17 Thread Kevin Sweeney 

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


Sorry Jay, I'll address your feedback in a subsequent review.

- Kevin Sweeney


On Oct. 16, 2014, 5:43 p.m., Kevin Sweeney wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26836/
> ---
> 
> (Updated Oct. 16, 2014, 5:43 p.m.)
> 
> 
> Review request for Aurora, Maxim Khutornenko and Bill Farner.
> 
> 
> Bugs: AURORA-440
> https://issues.apache.org/jira/browse/AURORA-440
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Cron documentation.
> 
> 
> Diffs
> -
> 
>   docs/configuration-reference.md 83d0d104554753dab939bf8d25b83e9adb00758a 
>   docs/cron-jobs.md PRE-CREATION 
>   examples/jobs/cron_hello_world.aurora PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/26836/diff/
> 
> 
> Testing
> ---
> 
> Pushed to 
> https://github.com/kevints/incubator-aurora/blob/kts/cron-docs/docs/cron-jobs.md
> 
> 
> Thanks,
> 
> Kevin Sweeney
> 
>

Re: Review Request 26836: Cron documentation

2014-10-17 Thread Jay Buffington 

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



docs/cron-jobs.md


I'd add a link here like this:

The full specification for cron_schedule is [described below(however you 
anchor link to "## Technical Note About Syntax")



docs/cron-jobs.md


s/indepdendently/independently/


- Jay Buffington


On Oct. 17, 2014, 12:43 a.m., Kevin Sweeney wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26836/
> ---
> 
> (Updated Oct. 17, 2014, 12:43 a.m.)
> 
> 
> Review request for Aurora, Maxim Khutornenko and Bill Farner.
> 
> 
> Bugs: AURORA-440
> https://issues.apache.org/jira/browse/AURORA-440
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Cron documentation.
> 
> 
> Diffs
> -
> 
>   docs/configuration-reference.md 83d0d104554753dab939bf8d25b83e9adb00758a 
>   docs/cron-jobs.md PRE-CREATION 
>   examples/jobs/cron_hello_world.aurora PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/26836/diff/
> 
> 
> Testing
> ---
> 
> Pushed to 
> https://github.com/kevints/incubator-aurora/blob/kts/cron-docs/docs/cron-jobs.md
> 
> 
> Thanks,
> 
> Kevin Sweeney
> 
>

Re: Review Request 26836: Cron documentation

2014-10-16 Thread Kevin Sweeney 

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

(Updated Oct. 16, 2014, 5:43 p.m.)


Review request for Aurora, Maxim Khutornenko and Bill Farner.


Changes
---

Update cron documentation.


Summary (updated)
-

Cron documentation


Bugs: AURORA-440
https://issues.apache.org/jira/browse/AURORA-440


Repository: aurora


Description (updated)
---

Cron documentation.


Diffs
-

  docs/configuration-reference.md 83d0d104554753dab939bf8d25b83e9adb00758a 
  docs/cron-jobs.md PRE-CREATION 
  examples/jobs/cron_hello_world.aurora PRE-CREATION 

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


Testing
---

Pushed to 
https://github.com/kevints/incubator-aurora/blob/kts/cron-docs/docs/cron-jobs.md


Thanks,

Kevin Sweeney