telemetry: Remove unnecessary Field constructor

We can just take the value parameter by value which allows both moving
into it, and copies at the same time, depending on the calling code.

src/common/telemetry.h

@@ -52,10 +52,7 @@ public:
 template <typename T>
 class Field : public FieldInterface {
 public:
-    Field(FieldType type, std::string name, const T& value)
-        : name(std::move(name)), type(type), value(value) {}
-
-    Field(FieldType type, std::string name, T&& value)
+    Field(FieldType type, std::string name, T value)
         : name(std::move(name)), type(type), value(std::move(value)) {}
 
     Field(const Field&) = default;