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.
---
