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.

Sign up for GitHub

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

Separates RegisterModelStep into RegisterLocalModelStep and RegisterRemoteModelStep. Adds GetMLTaskStep. Handles optional params #155

Merged
merged 13 commits into from
Nov 13, 2023

Adding todo for GetMLTask retry capability

cbd5af7
Select commit
Loading
Failed to load commit list.
Merged

Separates RegisterModelStep into RegisterLocalModelStep and RegisterRemoteModelStep. Adds GetMLTaskStep. Handles optional params #155

Adding todo for GetMLTask retry capability
cbd5af7
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 13, 2023 in 0s

90.37% of diff hit (target 70.00%)

View this Pull Request on Codecov

90.37% of diff hit (target 70.00%)

Annotations

Check warning on line 338 in src/main/java/org/opensearch/flowframework/indices/FlowFrameworkIndicesHandler.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/indices/FlowFrameworkIndicesHandler.java#L338 

Added line #L338 was not covered by tests

Check warning on line 94 in src/main/java/org/opensearch/flowframework/workflow/GetMLTaskStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/GetMLTaskStep.java#L94 

Added line #L94 was not covered by tests

Check warning on line 135 in src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java#L134-L135 

Added lines #L134 - L135 were not covered by tests

Check warning on line 167 in src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java#L167 

Added line #L167 was not covered by tests

Check warning on line 198 in src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java#L198 

Added line #L198 was not covered by tests

Check warning on line 126 in src/main/java/org/opensearch/flowframework/workflow/RegisterRemoteModelStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/RegisterRemoteModelStep.java#L126 

Added line #L126 was not covered by tests
View more details on Codecov