Spec edits for @defer/@stream#742

Closed

robrichard wants to merge 62 commits into

graphql:mainfrom

robrichard:incremental

Commits on Aug 31, 2023

Introduce @defer and @stream.
robrichard
committed
fix typos
robrichard
committed
clear up that it is legal to support either defer or stream individually
robrichard
committed
Add sumary of arguments to Type System
robrichard
committed
Update Section 3 -- Type System.md
robrichard
committed
clarification on defer/stream requirement
robrichard
committed
clarify negative values of initialCount
robrichard
committed
allow extensions only subsequent payloads
robrichard
committed
fix typo
robrichard
committed
Raise a field error if initialCount is less than zero
robrichard
committed
data is not necessarily an object in subsequent payloads
robrichard
committed
add Defer And Stream Directives Are Used On Valid Root Field rule
robrichard
committed
wait for parent async record to ensure correct order of payloads
robrichard
committed
Simplify execution, payloads should begin execution immediately
robrichard
committed
Clarify error handling
robrichard
committed
add isCompletedIterator to AsyncPayloadRecord to track completed iterators
robrichard
committed
fix typo
robrichard
committed
deferDirective and visitedFragments
robrichard
committed
stream if argument, indexPath -> itemPath
robrichard
committed
Clarify stream only applies to outermost list of multi-dimensional arrays
robrichard
committed
add validation “Defer And Stream Directive Labels Are Unique”
robrichard
committed
Clarification on labels
robrichard
committed
fix wrong quotes
robrichard
committed
remove label/path requirement
robrichard
committed
add missing line
robrichard
committed
fix ExecuteRequest
robrichard
committed
fix response
robrichard
committed
Align deferred fragment field collection with reference implementation
robrichard
committed
spec updates to reflect latest discussions
robrichard
committed
Note about mutation execution order
robrichard
committed
minor change for uniqueness
robrichard
committed
fix typos
robrichard
committed
if: Boolean! = true
robrichard
committed
address pr feedback
robrichard
committed
clarify null behavior of if
robrichard
committed
Add error boundary behavior
robrichard
committed
defer/stream response => payload
robrichard
committed
event stream => response stream
robrichard
committed
link to path section
robrichard
committed
use case no dash
robrichard
committed
remove "or null"
robrichard
committed
add detailed incremental example
robrichard
committed
update label validation rule
robrichard
committed
clarify hasNext on incremental example
robrichard
committed
clarify canceling of subsequent payloads
robrichard
committed
Add examples for non-null cases
robrichard
committed
typo
robrichard
committed
improve non-null example
robrichard
committed
Add FilterSubsequentPayloads algorithm
robrichard
committed
link to note on should
robrichard
committed
update on hasNext
robrichard
committed
small fixes (#3)

yaacovCR
authored and
robrichard
committed
remove ResolveFIeldGenerator (#4)

yaacovCR
authored and
robrichard
committed
fix typos (#6)

yaacovCR
authored and
robrichard
committed
Add error handling for stream iterators (#5)

yaacovCR
authored and
robrichard
committed
Raise a field error if defer/stream encountered during subscription execution
robrichard
committed
Add validation rule for defer/stream on subscriptions
robrichard
committed
clarify label is not required
robrichard
committed
fix parentRecord argument in ExecuteStreamField (#7)

yaacovCR
authored and
robrichard
committed
fix typo

robrichard
and
simongellis-attentive
committed
replace server with service
robrichard
committed
CollectFields does not require path or asyncRecord (#11)

yaacovCR
authored and
robrichard
committed