CV2-6728: N+1 Query (DynamicAnnotation::Field)

[melsawy]

Description

N+1 query occurs when run this GraphQL query

query { 
  # ID for verification_status Annotation
  node(id: "QW5ub3RhdGlvbi80NDE0OTYzNw==\n") { 
    ...F0 
  } 
}
fragment F0 on Annotation { 
  content
}

Calling to_json on the fields here triggers an N+1 query because each DynamicAnnotation::Field belongs to a field. During serialization, to_json performs a separate lookup for each associated FieldInstance. Using includes(:field_instance) eagerly loads the association, reducing the repeated queries to a single batched query.

References: CV2-6728

How to test?

Re-run unit tests

Checklist

• I have performed a self-review of my code and ensured that it is safe and runnable, that code coverage has not decreased, and that there are no new Code Climate issues. I have also followed Meedan's internal coding guidelines.