----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30957/#review72282 -----------------------------------------------------------
This patch does not apply cleanly on master (b62ec61), do you need to rebase? I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Feb. 13, 2015, 12:13 a.m., Kevin Sweeney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30957/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2015, 12:13 a.m.) > > > Review request for Aurora, Joshua Cohen and Bill Farner. > > > Repository: aurora > > > Description > ------- > > Extract ReadOnlyScheduler to its own implementation class and delegate from > the existing SchedulerThriftInterface to it. > > This enables serving ReadOnlyScheduler from a separate HTTP endpoint (e.g. an > unauthenticated one). > > This is almost a pure tool-driven refactor-rename change. Also renames Util > to Responses for improved ergonomics. > > To boost confidence in this change tests have been left in place. I will > follow-up with a subsequent review to split out ReadOnlySchedulerImplTest. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/http/api/ApiBeta.java > 53ea03bac3784baebc630ad1ce235d263985cafe > src/main/java/org/apache/aurora/scheduler/thrift/ReadOnlySchedulerImpl.java > PRE-CREATION > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > 8c19f3b08135eb5f3098591ebf9931b42a086318 > src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java > 7b28eb87767d7cd19e9365e3287f3e943d87dea5 > src/main/java/org/apache/aurora/scheduler/thrift/Util.java > 55242d18e08ea5cb6dd297bd7f18744d952580b3 > > src/main/java/org/apache/aurora/scheduler/thrift/aop/FeatureToggleInterceptor.java > e176a0df3141dcb088e05203cca4de3f0d3feea7 > > src/main/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptor.java > cad63c77e8144bb64c6b2acaf1b9199be13e4145 > > src/main/java/org/apache/aurora/scheduler/thrift/aop/UserCapabilityInterceptor.java > 5e6577d64b1037c69c3a952008240dcafe3b9f94 > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > 03d1fba76c23570c2c4102a48daf5ce035ecaaa3 > > Diff: https://reviews.apache.org/r/30957/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Kevin Sweeney > >