Github user jdasch commented on a diff in the pull request: https://github.com/apache/incubator-rya/pull/248#discussion_r148271883 --- Diff: extras/periodic.notification/service/src/main/java/org/apache/rya/periodic/notification/application/PeriodicNotificationApplicationConfiguration.java --- @@ -85,170 +85,170 @@ public PeriodicNotificationApplicationConfiguration(Properties props) { setPrunerThreads(Integer.parseInt(props.getProperty(PRUNER_THREADS, "1"))); setCoordinatorThreads(Integer.parseInt(props.getProperty(COORDINATOR_THREADS, "1"))); } - + /** * Sets the name of the Fluo Application - * @param fluoAppName + * @param fluoAppName */ - public void setFluoAppName(String fluoAppName) { - set(FLUO_APP_NAME, Preconditions.checkNotNull(fluoAppName)); + public void setFluoAppName(final String fluoAppName) { + set(FLUO_APP_NAME, Objects.requireNonNull(fluoAppName)); --- End diff -- I generally prefer to leave the class name in. For only 7 characters you get reduced ambiguity and increased readability.
---