Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/#review121951 --- Master (98a2bc1) is red with this patch.

Review Request 44368: Dropping bulkLoad() from Storage

2016-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44368/ --- Review request for Aurora, Joshua Cohen and Zameer Manji. Bugs: AURORA-1324

Re: Review Request 44368: Dropping bulkLoad() from Storage

2016-03-03 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44368/#review121958 --- Ship it! Ship It! - Joshua Cohen On March 3, 2016, 11:28

Re: Review Request 44368: Dropping bulkLoad() from Storage

2016-03-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44368/#review121959 --- Ship it! Ship It! - Zameer Manji On March 3, 2016, 3:28

Re: Review Request 44368: Dropping bulkLoad() from Storage

2016-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44368/#review121963 --- Ship it! Ship It! - Bill Farner On March 3, 2016, 3:28

Re: Review Request 44368: Dropping bulkLoad() from Storage

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44368/#review121962 --- Ship it! Master (98a2bc1) is green with this patch.

Re: Review Request 43622: Updating scheduler backup restore instructions.

2016-03-03 Thread Maxim Khutornenko
> On Feb. 23, 2016, 4:44 p.m., Stephan Erb wrote: > > docs/storage-config.md, line 99 > > > > > > I'd propose to move this section after the point "Identify the leading > > scheduler". Only at that point it is clear

Re: Review Request 44347: Install thrift package from the offical debs.

2016-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44347/#review121993 --- Ship it! Ship It! - Bill Farner On March 3, 2016, 10:56

Re: Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121984 --- I think the test failure is because subject is a mock, so you

Re: Review Request 43622: Updating scheduler backup restore instructions.

2016-03-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43622/ --- (Updated March 4, 2016, 1:52 a.m.) Review request for Aurora, John Sirois and

Re: Review Request 43622: Updating scheduler backup restore instructions.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43622/#review121997 --- Ship it! Master (98a2bc1) is green with this patch.

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/#review121990 --- Code LGTM, only blocker for me is the NEWS. NEWS (line 8)

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Amol Deshmukh
> On March 3, 2016, 6:07 p.m., Bill Farner wrote: > > NEWS, line 9 > > > > > > This is a kind of lackluster news entry. What does this actually > > _mean_ for users? Will i see different behavior? Is there a new

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/#review122021 --- Ship it! Master (3a3415e) is green with this patch.

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Amol Deshmukh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/ --- (Updated March 3, 2016, 10:14 p.m.) Review request for Aurora, Maxim

Review Request 44341: Updating documentation Update configuration-reference.md documentation on supporting private docker registry.

2016-03-03 Thread krish i
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44341/ --- Review request for Aurora. Repository: aurora Description --- Updating

Re: Review Request 44341: Updating documentation Update configuration-reference.md documentation on supporting private docker registry.

2016-03-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44341/#review121873 --- Ship it! Ship It! - Bill Farner On March 3, 2016, 8:27

Re: Review Request 44341: Updating documentation Update configuration-reference.md documentation on supporting private docker registry.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44341/#review121875 --- Master (8f98e6e) is red with this patch.

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Amol Deshmukh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/#review121924 --- @ReviewBot retry - Amol Deshmukh On March 2, 2016, 10:53 a.m.,

Review Request 44347: Install thrift package from the offical debs.

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

Re: Review Request 44347: Install thrift package from the offical debs.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44347/#review121911 --- Ship it! Master (98a2bc1) is green with this patch.

Re: Review Request 44347: Install thrift package from the offical debs.

2016-03-03 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44347/#review121917 --- Ship it! Ship It! - Jake Farrell On March 3, 2016, 6:56

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Amol Deshmukh
> On March 2, 2016, 11:15 a.m., Aurora ReviewBot wrote: > > Master (8f98e6e) is red with this patch. > > ./build-support/jenkins/build.sh > > > >  self._clock.tick(0.5 + epsilon) > >  > >

Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/ --- Review request for Aurora and Stephan Erb. Bugs: AURORA-1628

Re: Review Request 43925: AURORA-1616: [part 2] make tier_config mandatory argument when starting up the scheduler.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43925/#review121929 --- Master (98a2bc1) is red with this patch.

Re: Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121934 --- Ship it! Ship It! - Stephan Erb On März 3, 2016, 9:16

Re: Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121935 --- Is the build failure related? Code change itself looks fine (thus

Re: Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121941 --- @ReviewBot retry - Zameer Manji On March 3, 2016, 12:16 p.m.,

Re: Review Request 44354: Show principal in auth failure messaage.

2016-03-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121943 --- Master (98a2bc1) is red with this patch.