deferred fragments should not change ordering of fields within initial response#1054

Closed

yaacovCR wants to merge 64 commits intographql:mainfrom

yaacovCR:deduplicate4

Commits on Nov 18, 2023

Introduce @defer and @stream.

robrichard
authored and
yaacovCR
committed
fix typos

robrichard
authored and
yaacovCR
committed
clear up that it is legal to support either defer or stream individually

robrichard
authored and
yaacovCR
committed
Add sumary of arguments to Type System

robrichard
authored and
yaacovCR
committed
Update Section 3 -- Type System.md

robrichard
authored and
yaacovCR
committed
clarification on defer/stream requirement

robrichard
authored and
yaacovCR
committed
clarify negative values of initialCount

robrichard
authored and
yaacovCR
committed
allow extensions only subsequent payloads

robrichard
authored and
yaacovCR
committed
fix typo

robrichard
authored and
yaacovCR
committed
Raise a field error if initialCount is less than zero

robrichard
authored and
yaacovCR
committed
data is not necessarily an object in subsequent payloads

robrichard
authored and
yaacovCR
committed
add Defer And Stream Directives Are Used On Valid Root Field rule

robrichard
authored and
yaacovCR
committed
wait for parent async record to ensure correct order of payloads

robrichard
authored and
yaacovCR
committed
Simplify execution, payloads should begin execution immediately

robrichard
authored and
yaacovCR
committed
Clarify error handling

robrichard
authored and
yaacovCR
committed
add isCompletedIterator to AsyncPayloadRecord to track completed iterators

robrichard
authored and
yaacovCR
committed
fix typo

robrichard
authored and
yaacovCR
committed
deferDirective and visitedFragments

robrichard
authored and
yaacovCR
committed
stream if argument, indexPath -> itemPath

robrichard
authored and
yaacovCR
committed
Clarify stream only applies to outermost list of multi-dimensional arrays

robrichard
authored and
yaacovCR
committed
add validation “Defer And Stream Directive Labels Are Unique”

robrichard
authored and
yaacovCR
committed
Clarification on labels

robrichard
authored and
yaacovCR
committed
fix wrong quotes

robrichard
authored and
yaacovCR
committed
remove label/path requirement

robrichard
authored and
yaacovCR
committed
add missing line

robrichard
authored and
yaacovCR
committed
fix ExecuteRequest

robrichard
authored and
yaacovCR
committed
fix response

robrichard
authored and
yaacovCR
committed
Align deferred fragment field collection with reference implementation

robrichard
authored and
yaacovCR
committed
spec updates to reflect latest discussions

robrichard
authored and
yaacovCR
committed
Note about mutation execution order

robrichard
authored and
yaacovCR
committed
minor change for uniqueness

robrichard
authored and
yaacovCR
committed
fix typos

robrichard
authored and
yaacovCR
committed
if: Boolean! = true

robrichard
authored and
yaacovCR
committed
address pr feedback

robrichard
authored and
yaacovCR
committed
clarify null behavior of if

robrichard
authored and
yaacovCR
committed
Add error boundary behavior

robrichard
authored and
yaacovCR
committed
defer/stream response => payload

robrichard
authored and
yaacovCR
committed
event stream => response stream

robrichard
authored and
yaacovCR
committed
link to path section

robrichard
authored and
yaacovCR
committed
use case no dash

robrichard
authored and
yaacovCR
committed
remove "or null"

robrichard
authored and
yaacovCR
committed
add detailed incremental example

robrichard
authored and
yaacovCR
committed
update label validation rule

robrichard
authored and
yaacovCR
committed
clarify hasNext on incremental example

robrichard
authored and
yaacovCR
committed
clarify canceling of subsequent payloads

robrichard
authored and
yaacovCR
committed
Add examples for non-null cases

robrichard
authored and
yaacovCR
committed
typo

robrichard
authored and
yaacovCR
committed
improve non-null example

robrichard
authored and
yaacovCR
committed
Add FilterSubsequentPayloads algorithm

robrichard
authored and
yaacovCR
committed
link to note on should

robrichard
authored and
yaacovCR
committed
update on hasNext

robrichard
authored and
yaacovCR
committed
small fixes (#3)
yaacovCR
committed
remove ResolveFIeldGenerator (#4)
yaacovCR
committed
fix typos (#6)
yaacovCR
committed
Add error handling for stream iterators (#5)
yaacovCR
committed
Raise a field error if defer/stream encountered during subscription execution

robrichard
authored and
yaacovCR
committed
Add validation rule for defer/stream on subscriptions

robrichard
authored and
yaacovCR
committed
clarify label is not required

robrichard
authored and
yaacovCR
committed
fix parentRecord argument in ExecuteStreamField (#7)
yaacovCR
committed
fix typo

authored and
yaacovCR
committed
replace server with service

robrichard
authored and
yaacovCR
committed
CollectFields does not require path or asyncRecord (#11)
yaacovCR
committed

Commits on Nov 29, 2023

incremental delivery with deduplication of delivery and execution, allowing early delivery
yaacovCR
committed

Commits on Dec 1, 2023

deferred fields should not change ordering of non-deferred
yaacovCR
committed