Azure / LogicAppsUX

https://learn.microsoft.com/azure/logic-apps
MIT License
77 stars 83 forks source link

Description of concurrency control should match with what is the behavior and with v1 experience #5452

Open takyyon opened 2 months ago

takyyon commented 2 months ago

Describe the Bug with repro steps

This my sample flow:

image

Array variable value:

image

Compose action:

image

Delay action:

Delay by 10 seconds.

Test results for V3 designer:

Scenario 1: Keep default setting: "Concurrency control" = off

Loop 1:

image

Loop 2 is 10 seconds later:

image

Loop 3 is 10 seconds later:

image

Conclusion: it's executed sequentially.

Scenario 2: flip setting: "Concurrent control" = on and Degree of parallelism is 20:

image

Loop1:

image

Loop2:

image

Loop3:

image

Conclusion: it's executed in parallel.

Repeat the same scenarios in v1 designer, and got same results.

Verdict:

In the default setting, "Concurrent control" is off, LA runtime executes actions sequentially;

When toggling "Concurrent control" on and set a Degree of parallelism to larger than 1, LA runtime executes actions in parallel.

It's a runtime behavior, not a designer behavior.

V1 description matches runtime behavior:

image

V3 description is opposite of runtime behavior. This is a v3 LA designer bug.

image

Concurrency control description should be same as v1

What type of Logic App Is this happening in?

Consumption (Portal)

Which operating system are you using?

Windows

Are you using new designer or old designer

New Designer

Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg

Yes

Workflow JSON

No response

Screenshots or Videos

No response

Browser

[Chrome, Edge, Safari]

Additional context

No response

hartra344 commented 2 months ago

Discussed with Rohitha, and it looks like concurrency can be on a trigger and a for each and behavior of each is different. Our description describes how it works for Trigger. Rohitha is working with us to get messaging for For Each.