Github user meiercaleb commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/248#discussion_r148361952
--- Diff:
extras/periodic.notification/api/src/main/java/org/apache/rya/periodic/notification/api/PeriodicNotificationClient.java
---
@@ -36,29 +36,26 @@
* Adds a new notification to be registered with the {@link
NotificationCoordinatorExecutor}
* @param notification - notification to be added
*/
- public void addNotification(PeriodicNotification notification);
--- End diff --
Yeah, as a matter of style it's fine, but I'd prefer that you stay
consistent with the code base. Perhaps @amihalik could weigh in on this.
---
