Skip to content

Add Statestore Query to ExecutionService #97

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

Jump to bottom
Merged
YaroslavLitvinov merged 13 commits into from
Dec 24, 2025
Merged

Add Statestore Query to ExecutionService #97

YaroslavLitvinov merged 13 commits into from
Dec 24, 2025

Conversation

@YaroslavLitvinov
Copy link
Contributor

@YaroslavLitvinov YaroslavLitvinov commented Dec 23, 2025 •
edited
Loading

  • Rename QueryStatus -> ExecutionStatus, remove unneeded states
  • Prepare query's execution_status, error_code, result to be saved in Statestore
  • Executor unit test for StateStore query
  • Features
    • "state-store-query" feature to executor/embucketd/lambda
    • Add "traces-test-log" feature to api-snowflake-rest (to disable traces.log by default)
    • Add "tests" feature to crate/queries (to disable tests by default)
  • Fix build-info build script

@YaroslavLitvinov YaroslavLitvinov marked this pull request as draft December 23, 2025 00:44
@YaroslavLitvinov YaroslavLitvinov marked this pull request as ready for review December 24, 2025 02:40
@YaroslavLitvinov YaroslavLitvinov changed the title WIP: Add Statestore Query to ExecutionService Add Statestore Query to ExecutionService Dec 24, 2025
YaroslavLitvinov
YaroslavLitvinov commented Dec 24, 2025
View reviewed changes
crates/executor/src/tests/statestore_queries_unittest.rs

const TEST_SESSION_ID: &str = "test_session_id";

// it stucks without multithread
Copy link
Contributor Author

@YaroslavLitvinov YaroslavLitvinov Dec 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not related comment, will be removed eventually

@YaroslavLitvinov YaroslavLitvinov merged commit c7bcab5 into main Dec 24, 2025
3 checks passed
@YaroslavLitvinov YaroslavLitvinov deleted the yaro/statestore-mock branch December 24, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@osipovartem osipovartem osipovartem approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@YaroslavLitvinov @osipovartem