From ee15594070de39b743a69f05db078928c76bebda Mon Sep 17 00:00:00 2001 From: Andreas Huber Date: Tue, 11 Apr 2017 18:09:29 +0200 Subject: [PATCH] remove TODOs They don't make sense anymore. E.g. the Tags class is used by classes outside of org.lucares.performance.db. --- pdb-api/src/main/java/org/lucares/pdb/api/Tags.java | 2 -- pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java | 3 +-- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/pdb-api/src/main/java/org/lucares/pdb/api/Tags.java b/pdb-api/src/main/java/org/lucares/pdb/api/Tags.java index 8482b3d..b66595a 100644 --- a/pdb-api/src/main/java/org/lucares/pdb/api/Tags.java +++ b/pdb-api/src/main/java/org/lucares/pdb/api/Tags.java @@ -10,8 +10,6 @@ import java.util.TreeSet; import java.util.function.BiConsumer; public class Tags { - // TODO @ahr move class to org.lcuares.performance.db and make this package - // private public static final Tags EMPTY = new Tags(); private final Map tags; diff --git a/pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java b/pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java index 9854303..910ae25 100644 --- a/pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java +++ b/pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java @@ -141,8 +141,7 @@ public class TcpIngestor implements Ingestor, AutoCloseable, DisposableBean { // these fields are not tags break; case "tags": - // TODO @ahr add support for simple tags, currently we - // only support key/value tags + // ignore: we only support key/value tags break; default: tags = tags.copyAddIfNotNull(key, String.valueOf(value));