From 86d92d009189823aac86c162e9dd55c4ed7c48b0 Mon Sep 17 00:00:00 2001 From: Yusuke Tsutsumi Date: Sat, 19 Oct 2024 13:34:09 -0700 Subject: [PATCH 1/2] fix(lro): clarify protobuf response for no parallel requests For protobuf guidance, it was unclear response code should be. --- aep/general/0151/aep.md.j2 | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/aep/general/0151/aep.md.j2 b/aep/general/0151/aep.md.j2 index 42b42d5e..8a430d89 100644 --- a/aep/general/0151/aep.md.j2 +++ b/aep/general/0151/aep.md.j2 @@ -75,10 +75,23 @@ but is not obligated to do so: - Resources that accept multiple parallel requests **may** place them in a queue rather than work on the requests simultaneously. -- Resource that does not permit multiple requests in parallel (denying any new - request until the one that is in progress finishes) **must** return - `409 Conflict` if a user attempts a parallel request, and include an error - message explaining the situation. + +{% tab proto %} + +- A Resource that does not permit multiple requests in parallel (denying any new + request until the one that is in progress finishes) **must** return `ABORTED` + if a user attempts a parallel request, and include an error message explaining + the situation. + +{% tab oas %} + +- A Resource that does not permit multiple requests in parallel (denying any new + request until the one that is in progress finishes) **must** return a `409 + Conflict` if a user attempts a parallel request, and include an error message + explaining the situation. + +{% endtabs %} + ### Expiration From c6e71da613d96d1d2ea08c16e394be0e1e2660cc Mon Sep 17 00:00:00 2001 From: Yusuke Tsutsumi Date: Sun, 20 Oct 2024 21:12:29 -0700 Subject: [PATCH 2/2] addressing feedback --- aep/general/0151/aep.md.j2 | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/aep/general/0151/aep.md.j2 b/aep/general/0151/aep.md.j2 index 8a430d89..1374331b 100644 --- a/aep/general/0151/aep.md.j2 +++ b/aep/general/0151/aep.md.j2 @@ -78,21 +78,16 @@ but is not obligated to do so: {% tab proto %} -- A Resource that does not permit multiple requests in parallel (denying any new - request until the one that is in progress finishes) **must** return `ABORTED` - if a user attempts a parallel request, and include an error message explaining - the situation. +- A resource that does not support concurrent mutation requests **must** return an + `ABORTED` error if the resource has an outstanding mutation. {% tab oas %} -- A Resource that does not permit multiple requests in parallel (denying any new - request until the one that is in progress finishes) **must** return a `409 - Conflict` if a user attempts a parallel request, and include an error message - explaining the situation. +- A resource that does not support concurrent mutation requests **must** respond + with a `409 Conflict` status code if the resource has an outstanding mutation. {% endtabs %} - ### Expiration APIs **may** allow their operation resources to expire after sufficient time