Skip to content

Commit

Permalink
feat(query): add is_local to admin /v1/tables info (#16329)
Browse files Browse the repository at this point in the history
* feat(query): add is_local to admin /v1/tables info

* z
  • Loading branch information
everpcpc authored Aug 26, 2024
1 parent b5e6155 commit aaa653d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/reuse.sqllogic.yml
Original file line number Diff line number Diff line change
Expand Up @@ -112,19 +112,22 @@ jobs:
handler:
- "mysql"
- "http"
format:
- "native"
- "parquet"
steps:
- uses: actions/checkout@v4
- uses: ./.github/actions/test_sqllogic_standalone_linux_minio
timeout-minutes: 15
with:
dirs: ${{ matrix.dirs }}
handlers: ${{ matrix.handler }}
storage-format: all
storage-format: ${{ matrix.format }}
- name: Upload failure
if: failure()
uses: ./.github/actions/artifact_failure
with:
name: test-sqllogic-standalone-minio-${{ matrix.dirs }}-${{ matrix.handler }}
name: test-sqllogic-standalone-minio-${{ matrix.dirs }}-${{ matrix.handler }}-${{ matrix.format }}

cluster:
runs-on: [self-hosted, X64, Linux, 4c8g, "${{ inputs.runner_provider }}"]
Expand Down
2 changes: 2 additions & 0 deletions src/query/service/src/servers/admin/v1/tenant_tables.rs
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ pub struct TenantTableInfo {
pub engine: String,
pub created_on: DateTime<Utc>,
pub updated_on: DateTime<Utc>,
pub is_local: bool,
pub rows: u64,
pub data_bytes: u64,
pub compressed_data_bytes: u64,
Expand Down Expand Up @@ -106,6 +107,7 @@ async fn load_tenant_tables(tenant: &Tenant) -> Result<TenantTablesResponse> {
engine: table.engine().to_string(),
created_on: table.get_table_info().meta.created_on,
updated_on: table.get_table_info().meta.updated_on,
is_local: table.is_local(),
rows: stats.number_of_rows,
data_bytes: stats.data_bytes,
compressed_data_bytes: stats.compressed_data_bytes,
Expand Down

0 comments on commit aaa653d

Please sign in to comment.