From 1326e9fa5cffe326b82aeee9d82d008526aff947 Mon Sep 17 00:00:00 2001
From: Maros Marsalek <mmarsale@cisco.com>
Date: Mon, 5 Sep 2016 12:10:26 +0200
Subject: Cleanup TODOs and FIXMEs

- Fix minor ones
- Report bigger and include issue number in comment
- Pull common dependencies into dependency management of common/parents

Change-Id: I06a6ac37c52b603fd73ed42023d6b2e7fa18010f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
---
 infra/cfg-init/pom.xml                                               | 1 +
 .../java/io/fd/honeycomb/data/init/AbstractDataTreeConverter.java    | 5 +++--
 .../main/java/io/fd/honeycomb/data/init/InitializerRegistryImpl.java | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

(limited to 'infra/cfg-init')

diff --git a/infra/cfg-init/pom.xml b/infra/cfg-init/pom.xml
index df481b7da..c1fd353dc 100644
--- a/infra/cfg-init/pom.xml
+++ b/infra/cfg-init/pom.xml
@@ -39,6 +39,7 @@
             <artifactId>data-api</artifactId>
             <version>${project.version}</version>
         </dependency>
+
         <!-- Testing Dependencies -->
         <dependency>
             <groupId>junit</groupId>
diff --git a/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/AbstractDataTreeConverter.java b/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/AbstractDataTreeConverter.java
index 5476bb423..86188dd3f 100644
--- a/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/AbstractDataTreeConverter.java
+++ b/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/AbstractDataTreeConverter.java
@@ -97,9 +97,10 @@ public abstract class AbstractDataTreeConverter<O extends DataObject, C extends
         writeTx.submit().checkedGet();
     }
 
-    // TODO make this class concrete and use function dependency instead of abstract method
+    // TODO HONEYCOMB-158 make this class concrete and use function dependency instead of abstract method
     /**
-     * Converts operational data to config data for given root node
+     * Converts operational data to config data for given root node.
+     *
      * @param operationalData data object representing operational data
      * @return data object representing config data
      */
diff --git a/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/InitializerRegistryImpl.java b/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/InitializerRegistryImpl.java
index 2433130a9..cc1c2dbeb 100644
--- a/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/InitializerRegistryImpl.java
+++ b/infra/cfg-init/src/main/java/io/fd/honeycomb/data/init/InitializerRegistryImpl.java
@@ -35,7 +35,7 @@ public class InitializerRegistryImpl implements InitializerRegistry {
 
     @Override
     public void initialize() throws InitializeException {
-        // TODO check if readers are there
+        // TODO HONEYCOMB-159 check if readers are there
         LOG.debug("InitializerRegistryImpl.initialize()");
         for (DataTreeInitializer initializer : initializers) {
             initializer.initialize();
-- 
cgit 1.2.3-korg