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

Implemented throttling for max workflows to be created #151

Merged
merged 7 commits into from
Nov 13, 2023

Addressed PR comments

b388c7a
Select commit
Loading
Failed to load commit list.
Merged

Implemented throttling for max workflows to be created #151

Addressed PR comments
b388c7a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 9, 2023 in 0s

36.36% of diff hit (target 70.00%)

View this Pull Request on Codecov

36.36% of diff hit (target 70.00%)

Annotations

Check warning on line 96 in src/main/java/org/opensearch/flowframework/rest/RestCreateWorkflowAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/rest/RestCreateWorkflowAction.java#L96

Added line #L96 was not covered by tests

Check warning on line 114 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L114

Added line #L114 was not covered by tests

Check warning on line 120 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L116-L120

Added lines #L116 - L120 were not covered by tests

Check warning on line 125 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L123-L125

Added lines #L123 - L125 were not covered by tests

Check warning on line 131 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L127-L131

Added lines #L127 - L131 were not covered by tests

Check warning on line 133 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L133

Added line #L133 was not covered by tests

Check warning on line 135 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L135

Added line #L135 was not covered by tests

Check warning on line 137 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L137

Added line #L137 was not covered by tests

Check warning on line 140 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L140

Added line #L140 was not covered by tests

Check warning on line 144 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L144

Added line #L144 was not covered by tests

Check warning on line 147 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L147

Added line #L147 was not covered by tests

Check warning on line 150 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L149-L150

Added lines #L149 - L150 were not covered by tests

Check warning on line 152 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L152

Added line #L152 was not covered by tests

Check warning on line 154 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L154

Added line #L154 was not covered by tests

Check warning on line 200 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L199-L200

Added lines #L199 - L200 were not covered by tests

Check warning on line 202 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L202

Added line #L202 was not covered by tests

Check warning on line 204 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L204

Added line #L204 was not covered by tests

Check warning on line 206 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L206

Added line #L206 was not covered by tests

Check warning on line 208 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L208

Added line #L208 was not covered by tests

Check warning on line 214 in src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/CreateWorkflowTransportAction.java#L210-L214

Added lines #L210 - L214 were not covered by tests

Check warning on line 144 in src/main/java/org/opensearch/flowframework/transport/WorkflowRequest.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/WorkflowRequest.java#L144

Added line #L144 was not covered by tests

Check warning on line 152 in src/main/java/org/opensearch/flowframework/transport/WorkflowRequest.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/transport/WorkflowRequest.java#L152

Added line #L152 was not covered by tests