Review Request 37789: Refer to shared task_configs table for job updates.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37789/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-64

Review Request 37788: Deb: Clean up changelog generation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37788/ --- Review request for Aurora and Kevin Sweeney. Repository: aurora-packaging Des

Re: Review Request 37772: Fix RPM building.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37772/#review96494 --- Ship it! specs/rpm/aurora.spec (lines 62 - 72)

Re: Review Request 37772: Fix RPM building.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37772/ --- (Updated Aug. 25, 2015, 9:38 p.m.) Review request for Aurora and Maxim Khutorne

Re: Review Request 37772: Fix RPM building.

> On Aug. 25, 2015, 4:12 p.m., Kevin Sweeney wrote: > > specs/rpm/Makefile, lines 37-38 > > > > > > This is weird as a user will never be able to upgrade from a release > > version to a nightly. > > Maxim Khutornenk

Re: Review Request 37776: Setting revocable flag on a TaskInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37776/#review96480 --- Ship it! Ship It! - Bill Farner On Aug. 25, 2015, 4:05 p.m., Max

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96479 --- Ship it! Ship It! - David McLaughlin On Aug. 25, 2015, 3:01 p.m.

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? > > Joshua Cohen wrote: > I don't think we care, there's not much we can do in that case, is there? >

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? > > Joshua Cohen wrote: > I don't think we care, there's not much we can do in that case, is there? >

Re: Review Request 37776: Setting revocable flag on a TaskInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37776/#review96459 --- Ship it! Master (96a086c) is green with this patch. ./build-suppo

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? > > Joshua Cohen wrote: > I don't think we care, there's not much we can do in that case, is there? >

Re: Review Request 37772: Fix RPM building.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37772/#review96450 --- specs/rpm/Makefile (line 33)

Re: Review Request 37772: Fix RPM building.

> On Aug. 25, 2015, 11:12 p.m., Kevin Sweeney wrote: > > specs/rpm/Makefile, lines 37-38 > > > > > > This is weird as a user will never be able to upgrade from a release > > version to a nightly. Isn't version globa

Re: Review Request 37772: Fix RPM building.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37772/#review96443 --- specs/rpm/Makefile (lines 37 - 38)

Review Request 37776: Setting revocable flag on a TaskInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37776/ --- Review request for Aurora, Bill Farner and Zameer Manji. Repository: aurora D

Re: Review Request 37774: Adding TierManager binding into JMH benchmarks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37774/#review96439 --- Ship it! Ship It! - Bill Farner On Aug. 25, 2015, 3:16 p.m., Max

Re: Review Request 37719: Revocable: schema changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37719/#review96438 --- Ship it! Ship It! - Bill Farner On Aug. 25, 2015, 3:28 p.m., Max

Re: Review Request 37719: Revocable: schema changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37719/ --- (Updated Aug. 25, 2015, 10:28 p.m.) Review request for Aurora, Bill Farner and

Review Request 37774: Adding TierManager binding into JMH benchmarks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37774/ --- Review request for Aurora and Bill Farner. Repository: aurora Description ---

Review Request 37772: Fix RPM building.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37772/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-851 https://i

Re: Review Request 37770: Replace Twitter copyright headers in commons.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37770/#review96420 --- Ship it! Ship It! - Bill Farner On Aug. 25, 2015, 1:16 p.m., Zam

Review Request 37770: Replace Twitter copyright headers in commons.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37770/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-1442 https://issues

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? > > Joshua Cohen wrote: > I don't think we care, there's not much we can do in that case, is there? >

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? > > Joshua Cohen wrote: > I don't think we care, there's not much we can do in that case, is there? >

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

> On Aug. 25, 2015, 6:42 p.m., David McLaughlin wrote: > > What if they click on an old update and the instance page doesn't reflect > > the change made in this instance event? Do we care? I don't think we care, there's not much we can do in that case, is there? The only thing I can think of i

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96399 --- Master (86a547b) is green with this patch. ./build-support/jenkins

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96394 --- What if they click on an old update and the instance page doesn't re

Re: Review Request 37666: Import of Twitter Commons.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37666/ --- (Updated Aug. 25, 2015, 11:02 a.m.) Review request for Aurora, Jake Farrell, Ke

Re: Review Request 37666: Import of Twitter Commons.

> On Aug. 25, 2015, 8:18 a.m., Joshua Cohen wrote: > > .gitignore, lines 13-14 > > > > > > keep the contents of this file alpha-sorted? Done. > On Aug. 25, 2015, 8:18 a.m., Joshua Cohen wrote: > > build.gradle, lin

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96377 --- Ship it! Ship It! - Zameer Manji On Aug. 25, 2015, 8:01 a.m., Jo

Re: Review Request 37764: Update packages for post 0.9.0 changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37764/#review96374 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 25, 2015, 5:38 p.m

Re: Review Request 37764: Update packages for post 0.9.0 changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37764/ --- (Updated Aug. 25, 2015, 10:38 a.m.) Review request for Aurora and Maxim Khutorn

Review Request 37764: Update packages for post 0.9.0 changes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37764/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-851 https://i

Re: Review Request 37742: Remove use of host attributes from e2e tests.

> On Aug. 25, 2015, 3:31 a.m., Maxim Khutornenko wrote: > > src/test/sh/org/apache/aurora/e2e/http/http_example.aurora, lines 45-48 > > > > > > I am a bit concerned we will no longer have any constraint coverage in >

Re: Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/#review96362 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 25, 2015, 3:01 p.m

Re: Review Request 37666: Import of Twitter Commons.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37666/#review96339 --- Ship it! .gitignore (lines 13 - 14)

Review Request 37761: Add a link to the instance page from instance events on the update page.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37761/ --- Review request for Aurora, David McLaughlin and Zameer Manji. Bugs: AURORA-1331