Skip to content

Commit

Permalink
clarify errors from ExecuteExecutionPlan
Browse files Browse the repository at this point in the history
accompanying note is a WIP, open to further suggestions as to how to clarify
  • Loading branch information
yaacovCR committed Aug 26, 2024
1 parent 04d5803 commit 0058d2a
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion spec/Section 6 -- Execution.md
Original file line number Diff line number Diff line change
Expand Up @@ -335,7 +335,8 @@ serial):
- Let {data} and {incrementalDataRecords} be the result of
{ExecuteExecutionPlan(newDeferUsages, executionPlan, objectType, initialValue,
variableValues, serial)}.
- Let {errors} be the list of all _field error_ raised while completing {data}.
- Let {errors} be the list of all _field error_ raised while executing the
execution plan.
- If {incrementalDataRecords} is empty, return an unordered map containing
{data} and {errors}.
- Let {incrementalResults} be the result of {YieldIncrementalResults(data,
Expand All @@ -344,6 +345,9 @@ serial):
- Let {initialResult} be that result.
- Return {initialResult} and {BatchIncrementalResults(incrementalResults)}.

Note: {ExecuteExecutionPlan()} does not directly raise field errors from the
incremental portion of the Execution Plan.

### Yielding Incremental Results

The procedure for yielding incremental results is specified by the
Expand Down

0 comments on commit 0058d2a

Please sign in to comment.