----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30957/ -----------------------------------------------------------
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
