Removes unnecessary select and fixes potential time-of-check, time-of-use problem

[tempoz]

Rather than first performing a select to determine if we should insert or update, we can attempt an insert which does nothing on conflict and then update if the query executes without error but affects no rows.

Related issues: N/A

[tempoz]

• Remove unnecessary select in delete invocation with perms #6043 : 2 dependent PRs (#6044 , #6045 )
• Remove unnecessary select statement when updating targets #6014
• Uses a single query instead of two queries to retrieve a user by subid #6012
• Removes unnecessary select and fixes potential time-of-check, time-of-use problem #6010 👈
• Reduce telemetry insert to one query that does nothing on conflict #6003
• initial insert when registering invocation attempt doesn't need to be in a transaction #5865 : 1 other dependent PR (#5875 )
• Only perform the select to determine if there is no related execution row after attempting an update #6002
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tempoz and the rest of your teammates on Graphite

[tylerwilliams]

looks like this needs buildfix run on it or imports added

[tylerwilliams]

LGTM but please get @bduffany's approval too as this is auth code