Skip to content

Conversation

@YaroslavLitvinov
Copy link
Contributor

@YaroslavLitvinov YaroslavLitvinov commented Oct 23, 2025

Partial implementation of Metastore on sqlite

  • Volumes, databases, schemas are migrated on SQLITE (no vfs used)
  • We use diesel orm to map mentioned ^^ entities to sql.
  • Tables remained on slatedb, at least temporary.
  • ExecutionService::abort_query turned into async function as now it waits for abort confirmation

Known issues

  • UI should support some minor API changes in responses. Also didn't fix openapi spec, yet.
  • Slatedb sql view we use in execution engine renamed to sqlite view, and maybe need some fixes.
  • List of failing tests, probably due to some minor bugs in new Metastore implementation. Fixing them is a matter of time.
    tests::test_schemas
    tests::service::test_max_concurrency_level
    tests::service::test_query_recording
    tests::sql::commands::show::query_show_databases
    tests::sql::ddl::database::query_drop_database
    tests::sql::ddl::database::query_drop_database_error_in_use
    tests::test_generic_sqls::snowflake_generic::submit_ok_query_with_concurrent_limit
    tests::test_rest_quick_sqls::snowflake_compatibility::abort_ok_query
    tests::test_rest_quick_sqls::snowflake_compatibility::cancel_ok_query
    tests::test_rest_quick_sqls::snowflake_compatibility::cancel_ok_sleeping_query

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@YaroslavLitvinov YaroslavLitvinov marked this pull request as draft October 24, 2025 13:50
@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@YaroslavLitvinov YaroslavLitvinov changed the title WIP: DO NOT MERGE sqlite metastore Metastore on sqlite Nov 6, 2025
@YaroslavLitvinov YaroslavLitvinov marked this pull request as ready for review November 6, 2025 17:31
///
const KEY_TABLE: &str = "tbl";

pub struct SlateDBMetastore {
Copy link
Contributor

Choose a reason for hiding this comment

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

Sqlite or SlateDB metastore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Slatedb still used for tables

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@YaroslavLitvinov YaroslavLitvinov marked this pull request as draft November 7, 2025 12:18
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

SLT Targeted Testing: No relevant SLT tests found for the changes in this PR. No testing required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants