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

(Updated Jan. 6, 2015, 7:56 p.m.)


Review request for Aurora and Kevin Sweeney.


Changes
-------

Fixed bug links.

-wfarner


Bugs: AURORA-454 and AURORA-976
    https://issues.apache.org/jira/browse/AURORA-454
    https://issues.apache.org/jira/browse/AURORA-976


Repository: aurora


Description
-------

While working on AURORA-454, realized we have a bigger problem with cron 
schedules killing non-cron jobs. Field AURORA-976, which is also addressed in 
this diff.


Diffs
-----

  src/main/java/org/apache/aurora/scheduler/base/Query.java 
a6ff14a69a32965a87a7a3a6d73ef53f5bdf75c0 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
6e3caa164e4b98907e1a87460aab3afb9be95ef9 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 8d41e7011344aa6a9ba39feacc166dd3c3e64227 

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


Testing
-------

./gradlew -Pq build

```
$ aurora cron schedule  devcluster/www-data/prod/hello 
aurora/examples/jobs/cron_hello_world.aurora 
 INFO] Registering job hello with cron
Error scheduling cron job devcluster/www-data/prod/hello:
        Job www-data/prod/hello already exists
    
$ aurora job create  devcluster/www-data/prod/hello 
aurora/examples/jobs/hello_world.aurora 
 INFO] Creating job hello
Job creation failed due to error:
        Job www-data/prod/hello already exists
```


Thanks,

Maxim Khutornenko

Reply via email to