Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get commits #2

Merged
merged 9 commits into from
Nov 7, 2021
Merged

Conversation

benjaminloerincz
Copy link
Owner

No description provided.

benjaminloerincz and others added 9 commits June 17, 2021 23:08
Add performance warning to Flow documentation
* Fix Performance issues with TriggerActionFlow

Fixes #56

- Created an abstract `FlowTriggerRecord` class to hold the records that are being processed during trigger execution
- Enable definition of that class at the `SObject_Trigger_Setting` level
- Use the index in the newList as a static "breadcrumb" to tie records updates back to the newRecord during before Insert and before Update contexts
- Updated readme

Before: `MS to perform "GetOldRecord" on 200 cases : 15861`
After: `MS to perform TriggerRecord flow update on 200 cases : 2444`
- Clean up all tests
- Add more descriptive test names
- Remove any ambiguity in access modifiers
- Remove unnecessary Test.start and Test.stop statements
- Fix erroneous date in new classes
- Small performance improvements to TriggerActionFlow.applyFlowValues
- Add a new TriggerActionFlowIdentifier to enable cascading before insert and before update flows.
- Use `triggerActionFlowIdentifier|newListIndex` as a key to decrease risk of race conditions and collision on simple index match
@benjaminloerincz benjaminloerincz merged commit b5d9dc8 into benjaminloerincz:master Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants