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

Adds Create Index Building block #38

Merged
merged 6 commits into from
Sep 25, 2023

Addressed PR Comments

44da4d5
Select commit
Loading
Failed to load commit list.
Merged

Adds Create Index Building block #38

Addressed PR Comments
44da4d5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Sep 22, 2023 in 1s

78.37% of diff hit (target 74.26%)

View this Pull Request on Codecov

78.37% of diff hit (target 74.26%)

Annotations

Check warning on line 52 in src/main/java/org/opensearch/flowframework/FlowFrameworkPlugin.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/FlowFrameworkPlugin.java#L50-L52

Added lines #L50 - L52 were not covered by tests

Check warning on line 79 in src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java#L79

Added line #L79 was not covered by tests

Check warning on line 93 in src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java#L92-L93

Added lines #L92 - L93 were not covered by tests

Check warning on line 101 in src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/CreateIndex/CreateIndexStep.java#L101

Added line #L101 was not covered by tests