workflow: Remove record ID and simplify record store

[andrewwormald]

The goal of this MR is to simplify workflow a bit. The goal is to reduce mental complexity by removing the ID field for Records. Originally Workflow considers workflow name + foreignID + runID as a unique key for each run. As time has gone on I have thought more and more around the scalability of UUID v4 and the the low collision probability that it has even at the quantillion range is only reaching 50% probability.

Considering this and the desire to simplify the data model I am proposing that the runID is made a globally unique identifier for a workflow run which also aligns it more to it's name of RunID instead of using a combination of workflow name + foreignID + runID.

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Checklist before requesting a review

• I have performed a self-review of my code
• I have added thorough tests, if appropriate.

New Feature Submissions:

1. Does your submission pass tests?
2. Did you install pre-commit hooks before committing?

Changes to Core Features:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your core changes, as applicable?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
92.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[calentang]

hi，that very great code for workflow，can you support release of this luno/workflow project，thanks!

[andrewwormald]

HI @calentang, thank you! We will start tracking versions going forward. Thanks for reaching out!

Please see: https://github.com/luno/workflow/releases