fix: remove extra code

Author: lavkesh

src/main/java/io/odpf/depot/message/field/FieldUtils.java

@@ -34,10 +34,10 @@ public static String convertToString(Object value) {
 
     public static String convertToStringForSpecialTypes(Object value, Function<Object, String> toStringFunc) {
         if (value instanceof Collection<?>) {
-            return "[" + ((Collection<?>) value)
+            return GSON.toJson(((Collection<?>) value)
                     .stream()
-                    .map(o -> ("\"" + toStringFunc.apply(o) + "\""))
-                    .collect(Collectors.joining(",")) + "]";
+                    .map(toStringFunc)
+                    .collect(Collectors.toList()));
         }
         return toStringFunc.apply(value);
     }

src/main/java/io/odpf/depot/message/proto/converter/fields/ProtoFieldFactory.java

@@ -12,11 +12,11 @@ public static ProtoField getField(Descriptors.FieldDescriptor descriptor, Object
         List<ProtoField> protoFields = Arrays.asList(
                 new DurationProtoField(descriptor, fieldValue),
                 new TimestampProtoField(descriptor, fieldValue),
+                new MapProtoField(descriptor, fieldValue),
                 new EnumProtoField(descriptor, fieldValue),
                 new ByteProtoField(descriptor, fieldValue),
                 new StructProtoField(descriptor, fieldValue),
-                new MessageProtoField(descriptor, fieldValue),
-                new MapProtoField(descriptor, fieldValue)
+                new MessageProtoField(descriptor, fieldValue)
         );
         Optional<ProtoField> first = protoFields
                 .stream()