API-26761 Optimize VBADocuments::UploadRemover job

[caseywilliams]

Summary

• This job is mostly well behaved, but occasionally (seven times in the past month, for example) SQL queries can time out, extending the length of the job beyond the accepted limits. See the DataDog stats on this job here.
• These changes attempt to reduce the frequency that this happens by reducing the number of SQL queries and limiting the set of columns retrieved.

Related issue(s)

• API-26761

Testing done

• Just updated specs

What areas of the site does it impact?

• Attempts to optimize the job that removes old Benefits Intake uploads from S3

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution
• Documentation has been updated (link to documentation)
• No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
• Feature/bug has a monitor built into Datadog or Grafana (if applicable)
• If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
• I added a screenshot of the developed feature

[mathisto]

Beautiful. Nice call on the SELECT and batching.