Re: Review Request 22065: Fix /cron endpoint.

2014-05-30 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22065/#review44341 --- Ship it! Ship It! - Suman Karumuri On May 30, 2014, 1:32 a.m.,

Re: Review Request 22008: Exclude .git from rsync copy in vagrant.

2014-05-30 Thread Mark Chu-Carroll
On May 29, 2014, 7:58 a.m., Mark Chu-Carroll wrote: Why? Right now, I do a lot of debugging of the client using vagrant. My typical workflow is: - Make the change in my git workspace. - Commit it to a branch. - vagrant ssh into the virtual cluster - git pull /vagrant mybranch

Re: Review Request 22008: Exclude .git from rsync copy in vagrant.

2014-05-30 Thread Bill Farner
On May 29, 2014, 11:58 a.m., Mark Chu-Carroll wrote: Why? Right now, I do a lot of debugging of the client using vagrant. My typical workflow is: - Make the change in my git workspace. - Commit it to a branch. - vagrant ssh into the virtual cluster - git pull /vagrant mybranch

Re: Review Request 22008: Exclude .git from rsync copy in vagrant.

2014-05-30 Thread Bill Farner
On May 29, 2014, 11:58 a.m., Mark Chu-Carroll wrote: Why? Right now, I do a lot of debugging of the client using vagrant. My typical workflow is: - Make the change in my git workspace. - Commit it to a branch. - vagrant ssh into the virtual cluster - git pull /vagrant mybranch

Re: Review Request 22008: Exclude .git from rsync copy in vagrant.

2014-05-30 Thread Mark Chu-Carroll
On May 29, 2014, 7:58 a.m., Mark Chu-Carroll wrote: Why? Right now, I do a lot of debugging of the client using vagrant. My typical workflow is: - Make the change in my git workspace. - Commit it to a branch. - vagrant ssh into the virtual cluster - git pull /vagrant mybranch

Re: Review Request 22082: Modify the way that config binding helpers get registered.

2014-05-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22082/#review44394 --- Mark - would you mind adding a committer to the People line? We

Re: Review Request 22082: Modify the way that config binding helpers get registered.

2014-05-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22082/ --- (Updated May 30, 2014, 2:09 p.m.) Review request for Aurora, Antoine

Re: Review Request 22065: Fix /cron endpoint.

2014-05-30 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22065/#review44396 --- Ship it! Ship It! - David McLaughlin On May 30, 2014, 1:32

Re: Review Request 22082: Modify the way that config binding helpers get registered.

2014-05-30 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22082/#review44399 --- Ship it! Ship It! - David McLaughlin On May 30, 2014, 6:09

Review Request 22092: Added a comment to .gitignore.

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

Re: Review Request 22032: first draft of a developing client v2 document.

2014-05-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22032/#review44415 --- docs/developing-aurora-client.md

Re: Review Request 22032: first draft of a developing client v2 document.

2014-05-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22032/ --- (Updated May 30, 2014, 3:40 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 22092: Added a comment to .gitignore.

2014-05-30 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22092/#review44417 --- It wasn't obvious, but in the last review i was subtly trying to

Re: Review Request 22032: first draft of a developing client v2 document.

2014-05-30 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22032/ --- (Updated May 30, 2014, 4:14 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 22094: Updating vagrant cleanup steps.

2014-05-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22094/ --- (Updated May 30, 2014, 9:08 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 22094: Updating vagrant cleanup steps.

2014-05-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22094/ --- (Updated May 30, 2014, 9:13 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 22008: Exclude .git from rsync copy in vagrant.

2014-05-30 Thread Maxim Khutornenko
On May 29, 2014, 11:58 a.m., Mark Chu-Carroll wrote: Why? Right now, I do a lot of debugging of the client using vagrant. My typical workflow is: - Make the change in my git workspace. - Commit it to a branch. - vagrant ssh into the virtual cluster - git pull /vagrant mybranch

Re: Review Request 21440: Implementing parallel updater

2014-05-30 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21440/ --- (Updated May 30, 2014, 10:08 p.m.) Review request for Aurora, Mark Chu-Carroll

Review Request 22097: Remove unused thrift flags from scheduler

2014-05-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22097/ --- Review request for Aurora, David McLaughlin and Maxim Khutornenko. Repository:

Re: Review Request 22032: first draft of a developing client v2 document.

2014-05-30 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22032/#review44436 --- Ship it! Ship It! - Kevin Sweeney On May 30, 2014, 1:14 p.m.,