Feature/issue 155 - Log feature ids on db write

[torimcd]

Github Issue: #155 Log feature ids on db write

Description

Log a few reach_ids and node_ids of a granule during the database write so that automated tests can verify the features written

Overview of work done

Update logging statements in load_data module to write 5 reach_id and node_id values from the granule being ingested

Overview of verification done

No changes to unit tests, only logging statements changed

Overview of integration done

Deployed to SIT, loaded a few granules and verified reach_id and node_ids are being written to the logs during the db write.

PR checklist:

• Linted
• Updated unit tests
• Updated changelog
• Integration testing

See Pull Request Review Checklist for pointers on reviewing this pull request

[nikki-t]

This looks good to me. I was initially thinking it would be good to log all node and reach identifiers but I think that would be too many to be useful. It would make it tough to load and parse the logs so I like that you are adding a few to be used for verification. Plus I think this will be covered by issue #71.

[torimcd]

Agreed, it completely overwhelms the logs to write them all out. By adding the granuleUR and ingest_time in #150 it will be easy to run a query against the db directly to get all the ids that were written during a given ingest. I think that's a better way to do the full reconciliation for #71 than trying to parse it all out of the logs.