----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21849/ -----------------------------------------------------------
(Updated May 27, 2014, 6:08 p.m.) Review request for Aurora, David McLaughlin and Maxim Khutornenko. Changes ------- Removed copyright year line from license header, matching changes since this review was first posted. Repository: aurora Description ------- By default, the PMD plugin includes the 'basic' ruleSet. Rulesets are collections of individual rules, the downside being that they're all or none. To disable one rule within a ruleset, for example, you need to copy the ruleset definition XML and edit it. I've done that here with two rulesets, by minimally editing them (added apache header, and commented out disabled rules). If you're interested, you can see more info at the official ruleset listing [1]. Most of the larger edits made are to adhere to the ConfusingTernary rule [2]. I also moved the isCron() function as it tripped a rule, and after a bout of deja vu i realized it recently suffered a complete copy-paste. It is now de-cloned and lives in SanitizedConfiguration. [1] http://pmd.sourceforge.net/pmd-5.1.1/rules/index.html [2] http://pmd.sourceforge.net/pmd-5.1.1/rules/java/design.html#ConfusingTernary Diffs (updated) ----- build.gradle 0dbaa61bc2b0558ecab55b831b95ab6e704767e6 config/pmd/design.xml PRE-CREATION config/pmd/logging-java.xml PRE-CREATION src/main/java/org/apache/aurora/auth/SessionValidator.java 6b0f42335a6cb74f31403c13da33037f1f699aeb src/main/java/org/apache/aurora/scheduler/Driver.java 15e90d40e05bea2023e4904ed958a36d0f17d2a8 src/main/java/org/apache/aurora/scheduler/DriverFactory.java 549ef114b813718c7da7218cccb8aca600ce9dae src/main/java/org/apache/aurora/scheduler/MesosSchedulerImpl.java f05b05af0200d0681fa3b5147bb73e8f83264ff8 src/main/java/org/apache/aurora/scheduler/MesosTaskFactory.java 06a12a18f6d3e51dafdfaeb2d07511f16a67e9aa src/main/java/org/apache/aurora/scheduler/SchedulerLifecycle.java 5c0f09f312a8d5328f3c57dbf96c2201e0a33aba src/main/java/org/apache/aurora/scheduler/async/OfferQueue.java e2492de432a13282d1bad3596026d81b08599cd4 src/main/java/org/apache/aurora/scheduler/async/TaskGroups.java 17ddfc9077cbde36d73fe5a33a6d3fda0f902551 src/main/java/org/apache/aurora/scheduler/base/AsyncUtil.java a56f03736d65f992542f89f6897723bb39d53a1f src/main/java/org/apache/aurora/scheduler/base/CommandUtil.java da06f123ffd2207fa14e9682ec43957db5ea90f7 src/main/java/org/apache/aurora/scheduler/base/Query.java a5350c8b2e0494dcaf0577c77ac919ea7bb3dd79 src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java f86a5b742b760f723b3873e949a3c398e0a98195 src/main/java/org/apache/aurora/scheduler/configuration/Resources.java 5a38479bfbe41d06dc5224b77cbaaee0814585af src/main/java/org/apache/aurora/scheduler/configuration/SanitizedConfiguration.java 7fc493069455291deee15c75774a3d7cd4f8aa92 src/main/java/org/apache/aurora/scheduler/cron/quartz/AuroraCronJob.java cc0813774736168f9a8b4c125c1efa94978d7340 src/main/java/org/apache/aurora/scheduler/cron/quartz/CronLifecycle.java 8f72a2d46ed8c9cb82ff1d376f9348b7d5e56cb7 src/main/java/org/apache/aurora/scheduler/cron/quartz/CronModule.java 0094c88694d3bca5de08e38f6fbf99877885b352 src/main/java/org/apache/aurora/scheduler/events/PubsubEvent.java 06a5f8f5ab34861d71b167e95fdfa819232d8a1d src/main/java/org/apache/aurora/scheduler/filter/SchedulingFilter.java d8daa6865db9fca9fda8d27a5b076484e2a3ba87 src/main/java/org/apache/aurora/scheduler/http/JerseyTemplateServlet.java c955640e8ad7a797ee322826b392816867e2395d src/main/java/org/apache/aurora/scheduler/http/LeaderRedirect.java 9cb85bb41410ee30530a05b19cd4c86adbf8585c src/main/java/org/apache/aurora/scheduler/http/Quotas.java b516470e5f56423671d6b9f98d4a185cb55eeccd src/main/java/org/apache/aurora/scheduler/http/Slaves.java 56a8ce18bcbfc3329ffa2792311573cb42fc0483 src/main/java/org/apache/aurora/scheduler/http/StructDump.java 7bf2fba46bc0ac5fe308bf3ca9f54fc0719631a3 src/main/java/org/apache/aurora/scheduler/local/IsolatedSchedulerModule.java a9d5f8f9ee7a5204afe6b5b36b99badadf21ba31 src/main/java/org/apache/aurora/scheduler/quota/QuotaCheckResult.java 396cecf2b357169b4124ef6547fcbaf02adfbcb6 src/main/java/org/apache/aurora/scheduler/state/LockManagerImpl.java 3390038d6e2e6934e5f3c18a887e5f5f399390e5 src/main/java/org/apache/aurora/scheduler/state/SchedulerCoreImpl.java 46446a9dbaec3a98c6df376372d5c5fc1981b9c2 src/main/java/org/apache/aurora/scheduler/storage/AttributeStore.java 616bdc4295e90660f5d47c5a2d6d95a50992ec47 src/main/java/org/apache/aurora/scheduler/storage/JobStore.java dcd0ab0a1f99f4a6c1bf08ab506ff489fe1bb1e5 src/main/java/org/apache/aurora/scheduler/storage/LockStore.java 4e2371c7fb210c79ceef91d559cd55553f88cea5 src/main/java/org/apache/aurora/scheduler/storage/QuotaStore.java 0270e584f361dfc29a3c58586e56f7337bcf96aa src/main/java/org/apache/aurora/scheduler/storage/ReadWriteLockManager.java c16f70d31a80cd3d6dd1f479629e063f21aada1c src/main/java/org/apache/aurora/scheduler/storage/SchedulerStore.java f6a992d10009bba6f071977d6364878941b91b8b src/main/java/org/apache/aurora/scheduler/storage/Storage.java 768a8215b46433c003dac5471b94542e4a22d0b0 src/main/java/org/apache/aurora/scheduler/storage/StorageBackfill.java d507666b2bf6b0eedc74ae810c2be79b8bc25f5f src/main/java/org/apache/aurora/scheduler/storage/TaskStore.java 40503b441cd450576684a7a9d300710c50138484 src/main/java/org/apache/aurora/scheduler/storage/backup/BackupModule.java 17b79c3e8855a3df74bd4644c7afb51b3529207f src/main/java/org/apache/aurora/scheduler/storage/backup/Recovery.java e3c09e0aea27ba864fd242bb4cdeecdd337fb1c4 src/main/java/org/apache/aurora/scheduler/storage/log/LogManager.java c31fe2de0589df65212816c2af79f98a864cd2c4 src/main/java/org/apache/aurora/scheduler/storage/log/LogStorage.java 899c61a2eaf2ddca51cb637dd4ee3784b8dd24fe src/main/java/org/apache/aurora/scheduler/storage/log/testing/LogOpMatcher.java 87a442b0b6213f47dbf7cab9b3abef2f72907122 src/main/java/org/apache/aurora/scheduler/storage/mem/MemAttributeStore.java 429691c3cd7c9583d7d55a946c74908acff21288 src/main/java/org/apache/aurora/scheduler/storage/mem/MemJobStore.java 962861de19b5fd3cea8600a24926876481a8ae10 src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 6f3ebd3191c766f28a6824d4c12c500a7aecf576 src/main/java/org/apache/aurora/scheduler/thrift/aop/FeatureToggleInterceptor.java 8eb52dd87a79d9c17426197a5460e2a49d988e13 Diff: https://reviews.apache.org/r/21849/diff/ Testing ------- ./gradlew build Thanks, Bill Farner