[GitHub] incubator-rya pull request #248: RYA-356 Added a Twill App for running the p...

[meiercaleb]

Github user meiercaleb commented on a diff in the pull request:

    https://github.com/apache/incubator-rya/pull/248#discussion_r148108793
  
    --- 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 --
    
    Static import might look a little cleaner here.


---