Addressed comments

Signed-off-by: Athish Pranav D <[email protected]>

Author: Athishpranav2003

src/flb_msgpack_append_message.c

@@ -93,7 +93,7 @@ int flb_msgpack_append_map_to_record(char **result_buffer,
     msgpack_object_kv *new_map_entries[1];
     msgpack_object_kv  message_entry;
     char              *modified_data_buffer;
-    int             modified_data_size;
+    int                modified_data_size;
     size_t             off = 0;
     int                i;
     int                result = FLB_MAP_NOT_MODIFIED;
@@ -123,10 +123,10 @@ int flb_msgpack_append_map_to_record(char **result_buffer,
 
     message_entry.val = unpacker.data;
     result = flb_msgpack_expand_map(base_object_buffer,
-                                        base_object_size,
-                                        new_map_entries, 1,
-                                        &modified_data_buffer,
-                                        &modified_data_size);
+                                    base_object_size,
+                                    new_map_entries, 1,
+                                    &modified_data_buffer,
+                                    &modified_data_size);
     if (result == 0) {
         result = FLB_MAP_EXPAND_SUCCESS;
         *result_buffer = modified_data_buffer;

tests/runtime/filter_parser.c

@@ -879,15 +879,12 @@ void flb_test_filter_parser_hash_value_field()
     wait_with_timeout(2000, &output); /* waiting flush and ensuring data flush */
     TEST_CHECK_(output != NULL, "Expected output to not be NULL");
     if (output != NULL) {
-        /* check original field is preserved */
+        /* check hash value field present */
         expected = "\"parsed\":{\"INT\":\"100\",\"FLOAT\":\"0.5\",\"BOOL\":\"true\",\"STRING\":\"This is an example\"}";
         TEST_CHECK_(strstr(output, expected) != NULL, "Expected output to contain '%s', got '%s'", expected, output);
         /* check fields were extracted */
         expected = "\"INT\":\"100\",\"FLOAT\":\"0.5\",\"BOOL\":\"true\",\"STRING\":\"This is an example\"";
         TEST_CHECK_(strstr(output, expected) != NULL, "Expected output to contain '%s', got '%s'", expected, output);
-        /* check other fields are preserved */
-        // expected = "\"log\":\"An example\"";
-        // TEST_CHECK_(strstr(output, expected) != NULL, "Expected output to contain '%s', got '%s'", expected, output);
         free(output);
     }