andi/perfdb
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Activity

remove TODOs

Browse Source 
They don't make sense anymore.
E.g. the Tags class is used by classes outside of
org.lucares.performance.db.
This commit is contained in:
Andreas Huber
2017-04-11 18:09:29 +02:00
parent cc7e461ebf
commit ee15594070
2 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
pdb-api/src/main/java/org/lucares/pdb/api/Tags.java
View File

@@ -10,8 +10,6 @@ import java.util.TreeSet;
import java.util.function.BiConsumer; import java.util.function.BiConsumer;
public class Tags { public class Tags {
// TODO @ahr move class to org.lcuares.performance.db and make this package
// private
public static final Tags EMPTY = new Tags(); public static final Tags EMPTY = new Tags();
private final Map<String, Tag> tags; private final Map<String, Tag> tags;

3
pdb-ui/src/main/java/org/lucares/pdbui/TcpIngestor.java
View File

@@ -141,8 +141,7 @@ public class TcpIngestor implements Ingestor, AutoCloseable, DisposableBean {
// these fields are not tags // these fields are not tags
break; break;
case "tags": case "tags":
// TODO @ahr add support for simple tags, currently we // ignore: we only support key/value tags
// only support key/value tags
break; break;
default: default:
tags = tags.copyAddIfNotNull(key, String.valueOf(value)); tags = tags.copyAddIfNotNull(key, String.valueOf(value));