AshharAhmadKhan opened a new pull request, #5898: URL: https://github.com/apache/fineract/pull/5898
## Description The journal entry aggregation job was grouping entries only by asset owner, leaving `originator_external_ids` always NULL in the summary table even though the column existed. This meant the Trial Balance report had to re-derive originator data from raw journal entries for every historical row already in the snapshot, adding unnecessary memory overhead. It also meant originator never appeared via the snapshot path even when one was attached to the loan. This PR fixes both problems: - Adds a `loan_originators` CTE to the aggregation job reader query and wires `originator_external_ids` through the full pipeline — DTO, JPA entity, and writer service — so it gets correctly persisted per summary row - Updates the Trial Balance Summary Report (MySQL + PostgreSQL) via migration `0236` so `summary_snapshot_baseline_data` reads the stored originator value directly instead of hardcoding NULL, letting the snapshot join cleanly on originator without re-hitting the mapping tables - Updates unit tests for the reader and writer service to cover the new field - Adds an integration test that runs the actual aggregation job end-to-end and verifies originator surfaces correctly via the snapshot path -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
