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
feat(client): interactive transactions #8384
Conversation
…ong-running-transactions
…ong-running-transactions
ea0c578
to
1d7d5da
Compare
…ong-running-transactions
…g-running-transactions
packages/client/src/__tests__/integration/happy/interactive-transactions/test.ts
Outdated
Show resolved
Hide resolved
headers: string, | ||
transactionId?: string, | ||
): Promise<string> | ||
sdlSchema(): Promise<string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't remember, I'll have to ask Julius
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can take care of this this in a separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As always awesome work @millsp 🎉
get<R>(path: string, headers?: Client.DispatchOptions['headers']) { | ||
return this.raw<R>('GET', path, headers) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
get<R>(path: string, headers?: Client.DispatchOptions['headers']) { | |
return this.raw<R>('GET', path, headers) | |
} | |
get<R>(endpoint: string, headers?: Client.DispatchOptions['headers']) { | |
return this.raw<R>('GET', endpoint, headers) | |
} |
to make it consistent with POST
): Promise<{ data: T; elapsed: number }> | ||
requestBatch<T>( | ||
headers?: QueryEngineRequestHeaders, | ||
numTry?: number, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought numTry
was not used from last deep dive, is it used anywhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(We can do that in a separate PR)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work ✨
closes #1844
closes #7956
closes #7786
closes #7399
closes #6705
closes #7326
Internal doc