Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

> On Nov. 9, 2015, 7:17 p.m., Bill Farner wrote: > > Dmitriy - your reviewboard profile doesn't include an e-mail address, which > > would land in the commit message on master. Can you add one so we can give > > proper authorship credit? Shoot, I just changed it since @uber.com originating em

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105722 --- Dmitriy - your reviewboard profile doesn't include an e-mail addres

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105720 --- Ship it! Ship It! - Bill Farner On Nov. 4, 2015, 5:26 p.m., Dmi

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105719 --- Ship it! Ship It! - Maxim Khutornenko On Nov. 5, 2015, 1:26 a.m

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105716 --- Ship it! Ship It! - Zameer Manji On Nov. 4, 2015, 5:26 p.m., Dm

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105201 --- Ship it! Master (745869c) is green with this patch. ./build-supp

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/ --- (Updated Nov. 5, 2015, 1:26 a.m.) Review request for Aurora, Maxim Khutornenko,

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105197 --- Master (745869c) is red with this patch. ./build-support/jenkins/

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/ --- (Updated Nov. 5, 2015, 1:08 a.m.) Review request for Aurora, Maxim Khutornenko,

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/ --- (Updated Nov. 5, 2015, 1:03 a.m.) Review request for Aurora, Maxim Khutornenko,

Re: Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/#review105163 --- This patch does not apply cleanly on master (745869c), do you need

Review Request 39958: deleting unused function deprecation_warning adding test coverage to base, job_monitor, quota_check, scheduler_client

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39958/ --- Review request for Aurora, Maxim Khutornenko, Bill Farner, and Zameer Manji. Bu