Skip to content
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

continue-on-error in batch requests #2038

Closed
HeikoTheissen opened this issue Dec 10, 2024 · 1 comment · Fixed by #2043
Closed

continue-on-error in batch requests #2038

HeikoTheissen opened this issue Dec 10, 2024 · 1 comment · Fixed by #2043

Comments

@HeikoTheissen
Copy link
Contributor

HeikoTheissen commented Dec 10, 2024

OData-Protocol, section 8.2.8.3 states that

The continue-on-error preference SHOULD NOT be applied to individual requests within a batch.

This could be understood to mean that there can be no continue-on-error behavior at all for batch requests. But what we want is that the continue-on-error preference of the batch request is applied to all individual requests, according to the specification for

Clarify this in the text.

@HeikoTheissen HeikoTheissen moved this to Open in OData TC Dec 10, 2024
@HeikoTheissen HeikoTheissen changed the title continue-on-error in $each requests in batch requests continue-on-error in batch requests Dec 12, 2024
@mikepizzo
Copy link
Contributor

The statement "The continue-on-error preference SHOULD NOT be applied to individual requests within a batch." should instead be that the continue-on-error specified on batch represents the processing of statements within the batch itself, not whether individual statements within the batch that represent multiple operations should continue on error.

@HeikoTheissen HeikoTheissen moved this from Open to Resolved in OData TC Jan 9, 2025
mikepizzo pushed a commit that referenced this issue Jan 15, 2025
@github-project-automation github-project-automation bot moved this from Resolved to Closed in OData TC Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

2 participants