Re: Review Request 22097: Remove unused thrift flags from scheduler

2014-06-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22097/#review44568 --- Please drop these as well: build.gradle 473:'-thrift_port=8082',

Re: Review Request 22097: Remove unused thrift flags from scheduler

2014-06-02 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22097/#review44553 --- Ship it! Ship It! - Maxim Khutornenko On May 30, 2014, 10:52 p.m

Re: Review Request 22097: Remove unused thrift flags from scheduler

2014-06-02 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22097/#review44549 --- Ship it! Code change lgtm, although I can't speak for the impact of