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

add InteractiveSession and SessionManager #2290

Merged

address comments

9693f31
Select commit
Loading
Failed to load commit list.
Merged

add InteractiveSession and SessionManager #2290

address comments
9693f31
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 13, 2023 in 0s

96.87% of diff hit (target 96.60%)

View this Pull Request on Codecov

96.87% of diff hit (target 96.60%)

Annotations

Check warning on line 81 in spark/src/main/java/org/opensearch/sql/spark/execution/session/SessionModel.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

spark/src/main/java/org/opensearch/sql/spark/execution/session/SessionModel.java#L81

Added line #L81 was not covered by tests

Check warning on line 60 in spark/src/main/java/org/opensearch/sql/spark/execution/statestore/SessionStateStore.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

spark/src/main/java/org/opensearch/sql/spark/execution/statestore/SessionStateStore.java#L59-L60

Added lines #L59 - L60 were not covered by tests

Check warning on line 84 in spark/src/main/java/org/opensearch/sql/spark/execution/statestore/SessionStateStore.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

spark/src/main/java/org/opensearch/sql/spark/execution/statestore/SessionStateStore.java#L83-L84

Added lines #L83 - L84 were not covered by tests