Skip to content

Commit 879818f

Browse files
yaacovCRrobrichard
authored andcommitted
clarify errors from ExecuteExecutionPlan
accompanying note is a WIP, open to further suggestions as to how to clarify
1 parent ccc26f2 commit 879818f

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

spec/Section 6 -- Execution.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,8 @@ serial):
338338
- Let {data} and {incrementalDataRecords} be the result of
339339
{ExecuteExecutionPlan(newDeferUsages, executionPlan, objectType, initialValue,
340340
variableValues, serial)}.
341-
- Let {errors} be the list of all _field error_ raised while completing {data}.
341+
- Let {errors} be the list of all _field error_ raised while executing the
342+
execution plan.
342343
- If {incrementalDataRecords} is empty, return an unordered map containing
343344
{data} and {errors}.
344345
- Let {incrementalResults} be the result of {YieldIncrementalResults(data,
@@ -347,6 +348,9 @@ serial):
347348
- Let {initialResult} be that result.
348349
- Return {initialResult} and {BatchIncrementalResults(incrementalResults)}.
349350

351+
Note: {ExecuteExecutionPlan()} does not directly raise field errors from the
352+
incremental portion of the Execution Plan.
353+
350354
### Yielding Incremental Results
351355

352356
The procedure for yielding incremental results is specified by the

0 commit comments

Comments
 (0)