fix: some cr comments

.cargo/config.toml

@@ -3,3 +3,13 @@ linker = "aarch64-linux-gnu-gcc"
 
 [alias]
 sqlness = "run --bin sqlness-runner --"
+
+
+[build]
+rustflags = [
+    # lints
+    # TODO: use lint configuration in cargo https://github.com/rust-lang/cargo/issues/5034
+    "-Wclippy::print_stdout",
+    "-Wclippy::print_stderr",
+    "-Wclippy::implicit_clone",
+]

.editorconfig

@@ -1,10 +0,0 @@
-root = true
-
-[*]
-end_of_line = lf
-indent_style = space
-insert_final_newline = true
-trim_trailing_whitespace = true
-
-[{Makefile,**.mk}]
-indent_style = tab

.env.example

@@ -21,6 +21,3 @@ GT_GCS_CREDENTIAL_PATH = GCS credential path
 GT_GCS_ENDPOINT = GCS end point
 # Settings for kafka wal test
 GT_KAFKA_ENDPOINTS = localhost:9092
-
-# Setting for fuzz tests
-GT_MYSQL_ADDR = localhost:4002

.github/actions/build-linux-artifacts/action.yml

@@ -34,7 +34,7 @@ runs:
 
     - name: Upload sqlness logs
       if: ${{ failure() && inputs.disable-run-tests == 'false' }} # Only upload logs when the integration tests failed.
-      uses: actions/upload-artifact@v4
+      uses: actions/upload-artifact@v3
       with:
         name: sqlness-logs
         path: /tmp/greptime-*.log

.github/actions/build-macos-artifacts/action.yml

@@ -67,7 +67,7 @@ runs:
 
     - name: Upload sqlness logs
       if: ${{ failure() }} # Only upload logs when the integration tests failed.
-      uses: actions/upload-artifact@v4
+      uses: actions/upload-artifact@v3
       with:
         name: sqlness-logs
         path: /tmp/greptime-*.log

.github/actions/build-windows-artifacts/action.yml

@@ -62,15 +62,15 @@ runs:
 
     - name: Upload sqlness logs
       if: ${{ failure() }} # Only upload logs when the integration tests failed.
-      uses: actions/upload-artifact@v4
+      uses: actions/upload-artifact@v3
       with:
         name: sqlness-logs
-        path: /tmp/greptime-*.log
+        path: ${{ runner.temp }}/greptime-*.log
         retention-days: 3
 
     - name: Build greptime binary
       shell: pwsh
-      run: cargo build --profile ${{ inputs.cargo-profile }} --features ${{ inputs.features }} --target ${{ inputs.arch }} --bin greptime
+      run: cargo build --profile ${{ inputs.cargo-profile }} --features ${{ inputs.features }} --target ${{ inputs.arch }}
 
     - name: Upload artifacts
       uses: ./.github/actions/upload-artifacts

.github/actions/fuzz-test/action.yaml

@@ -1,13 +0,0 @@
-name: Fuzz Test
-description: 'Fuzz test given setup and service'
-inputs:
-  target:
-    description: "The fuzz target to test"
-runs:
-  using: composite
-  steps:
-  - name: Run Fuzz Test
-    shell: bash
-    run: cargo fuzz run ${{ inputs.target }} --fuzz-dir tests-fuzz -D -s none -- -max_total_time=120
-    env:
-      GT_MYSQL_ADDR: 127.0.0.1:4002

.github/pull_request_template.md

@@ -1,10 +1,8 @@
-I hereby agree to the terms of the [GreptimeDB CLA](https://github.com/GreptimeTeam/.github/blob/main/CLA.md).
-
-## Refer to a related PR or issue link (optional)
+I hereby agree to the terms of the [GreptimeDB CLA](https://gist.github.com/xtang/6378857777706e568c1949c7578592cc)
 
 ## What's changed and what's your intention?
 
-__!!! DO NOT LEAVE THIS BLOCK EMPTY !!!__
+_PLEASE DO NOT LEAVE THIS EMPTY !!!_
 
 Please explain IN DETAIL what the changes are in this PR and why they are needed:
 
@@ -18,3 +16,5 @@ Please explain IN DETAIL what the changes are in this PR and why they are needed
 - [ ]  I have written the necessary rustdoc comments.
 - [ ]  I have added the necessary unit tests and integration tests.
 - [x]  This PR does not require documentation updates.
+
+## Refer to a related PR or issue link (optional)

.github/workflows/develop.yml

@@ -1,7 +1,7 @@
 on:
   merge_group:
   pull_request:
-    types: [ opened, synchronize, reopened, ready_for_review ]
+    types: [opened, synchronize, reopened, ready_for_review]
     paths-ignore:
       - 'docs/**'
       - 'config/**'
@@ -57,7 +57,7 @@ jobs:
           toolchain: ${{ env.RUST_TOOLCHAIN }}
       - name: Rust Cache
         uses: Swatinem/rust-cache@v2
-        with:
+        with: 
           # Shares across multiple jobs
           # Shares with `Clippy` job
           shared-key: "check-lint"
@@ -75,7 +75,7 @@ jobs:
           toolchain: stable
       - name: Rust Cache
         uses: Swatinem/rust-cache@v2
-        with:
+        with: 
           # Shares across multiple jobs
           shared-key: "check-toml"
       - name: Install taplo
@@ -102,7 +102,7 @@ jobs:
           shared-key: "build-binaries"
       - name: Build greptime binaries
         shell: bash
-        run: cargo build --bin greptime --bin sqlness-runner
+        run: cargo build
       - name: Pack greptime binaries
         shell: bash
         run: |
@@ -117,46 +117,6 @@ jobs:
           artifacts-dir: bins
           version: current
 
-  fuzztest:
-    name: Fuzz Test
-    needs: build
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        target: [ "fuzz_create_table", "fuzz_alter_table" ]
-    steps:
-      - uses: actions/checkout@v4
-      - uses: arduino/setup-protoc@v3
-      - uses: dtolnay/rust-toolchain@master
-        with:
-          toolchain: ${{ env.RUST_TOOLCHAIN }}
-      - name: Rust Cache
-        uses: Swatinem/rust-cache@v2
-        with:
-          # Shares across multiple jobs
-          shared-key: "fuzz-test-targets"
-      - name: Set Rust Fuzz
-        shell: bash
-        run: |
-          sudo apt update && sudo apt install -y libfuzzer-14-dev
-          cargo install cargo-fuzz
-      - name: Download pre-built binaries
-        uses: actions/download-artifact@v4
-        with:
-          name: bins
-          path: .
-      - name: Unzip binaries
-        run: tar -xvf ./bins.tar.gz
-      - name: Run GreptimeDB
-        run: |
-          ./bins/greptime standalone start&
-      - name: Fuzz Test
-        uses: ./.github/actions/fuzz-test
-        env:
-          CUSTOM_LIBFUZZER_PATH: /usr/lib/llvm-14/lib/libFuzzer.a
-        with:
-          target: ${{ matrix.target }}
-
   sqlness:
     name: Sqlness Test
     needs: build
@@ -176,12 +136,13 @@ jobs:
         run: tar -xvf ./bins.tar.gz
       - name: Run sqlness
         run: RUST_BACKTRACE=1 ./bins/sqlness-runner -c ./tests/cases --bins-dir ./bins
+      # FIXME: Logs cannot found be on failure (or even success). Need to figure out the cause.
       - name: Upload sqlness logs
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v3
         with:
           name: sqlness-logs
-          path: /tmp/greptime-*.log
+          path: ${{ runner.temp }}/greptime-*.log
           retention-days: 3
 
   sqlness-kafka-wal:
@@ -206,12 +167,13 @@ jobs:
         run: docker compose -f docker-compose-standalone.yml up -d --wait
       - name: Run sqlness
         run: RUST_BACKTRACE=1 ./bins/sqlness-runner -w kafka -k 127.0.0.1:9092 -c ./tests/cases --bins-dir ./bins
+      # FIXME: Logs cannot be found on failure (or even success). Need to figure out the cause.
       - name: Upload sqlness logs
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v3
         with:
-          name: sqlness-logs-with-kafka-wal
-          path: /tmp/greptime-*.log
+          name: sqlness-logs
+          path: ${{ runner.temp }}/greptime-*.log
           retention-days: 3
 
   fmt:
@@ -229,7 +191,7 @@ jobs:
           components: rustfmt
       - name: Rust Cache
         uses: Swatinem/rust-cache@v2
-        with:
+        with: 
           # Shares across multiple jobs
           shared-key: "check-rust-fmt"
       - name: Run cargo fmt
@@ -250,7 +212,7 @@ jobs:
           components: clippy
       - name: Rust Cache
         uses: Swatinem/rust-cache@v2
-        with:
+        with: 
           # Shares across multiple jobs
           # Shares with `Check` job
           shared-key: "check-lint"
@@ -279,10 +241,6 @@ jobs:
         with:
           # Shares cross multiple jobs
           shared-key: "coverage-test"
-      - name: Docker Cache
-        uses: ScribeMD/docker-cache@0.3.7
-        with:
-          key: docker-${{ runner.os }}-coverage
       - name: Install latest nextest release
         uses: taiki-e/install-action@nextest
       - name: Install cargo-llvm-cov
@@ -313,28 +271,10 @@ jobs:
           GT_KAFKA_ENDPOINTS: 127.0.0.1:9092
           UNITTEST_LOG_DIR: "__unittest_logs"
       - name: Codecov upload
-        uses: codecov/codecov-action@v4
+        uses: codecov/codecov-action@v2
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: ./lcov.info
           flags: rust
           fail_ci_if_error: false
           verbose: true
-
-  compat:
-    name: Compatibility Test
-    needs: build
-    runs-on: ubuntu-20.04
-    timeout-minutes: 60
-    steps:
-      - uses: actions/checkout@v4
-      - name: Download pre-built binaries
-        uses: actions/download-artifact@v4
-        with:
-          name: bins
-          path: .
-      - name: Unzip binaries
-        run: |
-          mkdir -p ./bins/current
-          tar -xvf ./bins.tar.gz --strip-components=1 -C ./bins/current
-      - run: ./tests/compat/test-compat.sh 0.6.0

.github/workflows/docs.yml

@@ -61,18 +61,6 @@ jobs:
 
   sqlness:
     name: Sqlness Test
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ ubuntu-20.04 ]
-    steps:
-      - run: 'echo "No action required"'
-
-  sqlness-kafka-wal:
-    name: Sqlness Test with Kafka Wal
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ ubuntu-20.04 ]
+    runs-on: ubuntu-20.04
     steps:
       - run: 'echo "No action required"'

.github/workflows/nightly-ci.yml

@@ -45,10 +45,10 @@ jobs:
             {"text": "Nightly CI failed for sqlness tests"}
       - name: Upload sqlness logs
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v3
         with:
           name: sqlness-logs
-          path: /tmp/greptime-*.log
+          path: ${{ runner.temp }}/greptime-*.log
           retention-days: 3
 
   test-on-windows:

.github/workflows/release.yml

@@ -91,7 +91,7 @@ env:
   # The scheduled version is '${{ env.NEXT_RELEASE_VERSION }}-nightly-YYYYMMDD', like v0.2.0-nigthly-20230313;
   NIGHTLY_RELEASE_PREFIX: nightly
   # Note: The NEXT_RELEASE_VERSION should be modified manually by every formal release.
-  NEXT_RELEASE_VERSION: v0.8.0
+  NEXT_RELEASE_VERSION: v0.7.0
 
 jobs:
   allocate-runners:
@@ -221,8 +221,6 @@ jobs:
             arch: x86_64-apple-darwin
             artifacts-dir-prefix: greptime-darwin-amd64-pyo3
     runs-on: ${{ matrix.os }}
-    outputs:
-      build-macos-result: ${{ steps.set-build-macos-result.outputs.build-macos-result }}
     needs: [
       allocate-runners,
     ]
@@ -262,8 +260,6 @@ jobs:
             features: pyo3_backend,servers/dashboard
             artifacts-dir-prefix: greptime-windows-amd64-pyo3
     runs-on: ${{ matrix.os }}
-    outputs:
-      build-windows-result: ${{ steps.set-build-windows-result.outputs.build-windows-result }}
     needs: [
       allocate-runners,
     ]
@@ -288,7 +284,7 @@ jobs:
       - name: Set build windows result
         id: set-build-windows-result
         run: |
-          echo "build-windows-result=success" >> $Env:GITHUB_OUTPUT
+          echo "build-windows-result=success" >> $GITHUB_OUTPUT    
 
   release-images-to-dockerhub:
     name: Build and push images to DockerHub
@@ -299,8 +295,6 @@ jobs:
       build-linux-arm64-artifacts,
     ]
     runs-on: ubuntu-2004-16-cores
-    outputs:
-      build-image-result: ${{ steps.set-build-image-result.outputs.build-image-result }}
     steps:
       - uses: actions/checkout@v4
         with:
@@ -316,7 +310,7 @@ jobs:
           version: ${{ needs.allocate-runners.outputs.version }}
 
       - name: Set build image result
-        id: set-build-image-result
+        id: set-image-build-result
         run: |
           echo "build-image-result=success" >> $GITHUB_OUTPUT    
 

.github/workflows/unassign.yml

@@ -1,21 +0,0 @@
-name: Auto Unassign
-on:
-  schedule:
-    - cron: '4 2 * * *'
-  workflow_dispatch:
-
-permissions:
-  contents: read
-  issues: write
-  pull-requests: write
-
-jobs:
-  auto-unassign:
-    name: Auto Unassign
-    runs-on: ubuntu-latest
-    steps:
-      - name: Auto Unassign
-        uses: tisonspieces/auto-unassign@main
-        with:
-          token: ${{ secrets.GH_PERSONAL_ACCESS_TOKEN }}
-          repository: ${{ github.repository }}

.gitignore

@@ -46,7 +46,3 @@ benchmarks/data
 *.code-workspace
 
 venv/
-
-# Fuzz tests 
-tests-fuzz/artifacts/
-tests-fuzz/corpus/

Cargo.toml

@@ -18,7 +18,6 @@ members = [
     "src/common/grpc-expr",
     "src/common/mem-prof",
     "src/common/meta",
-    "src/common/plugins",
     "src/common/procedure",
     "src/common/procedure-test",
     "src/common/query",
@@ -62,23 +61,17 @@ members = [
 resolver = "2"
 
 [workspace.package]
-version = "0.7.1"
+version = "0.6.0"
 edition = "2021"
 license = "Apache-2.0"
 
-[workspace.lints]
-clippy.print_stdout = "warn"
-clippy.print_stderr = "warn"
-clippy.implicit_clone = "warn"
-rust.unknown_lints = "deny"
-
 [workspace.dependencies]
 ahash = { version = "0.8", features = ["compile-time-rng"] }
 aquamarine = "0.3"
 arrow = { version = "47.0" }
 arrow-array = "47.0"
 arrow-flight = "47.0"
-arrow-ipc = { version = "47.0", features = ["lz4"] }
+arrow-ipc = "47.0"
 arrow-schema = { version = "47.0", features = ["serde"] }
 async-stream = "0.3"
 async-trait = "0.1"
@@ -103,11 +96,11 @@ etcd-client = "0.12"
 fst = "0.4.7"
 futures = "0.3"
 futures-util = "0.3"
-greptime-proto = { git = "https://github.com/GreptimeTeam/greptime-proto.git", rev = "349cb385583697f41010dabeb3c106d58f9599b4" }
+greptime-proto = { git = "https://github.com/GreptimeTeam/greptime-proto.git", rev = "96f1f0404f421ee560a4310c73c5071e49168168" }
 humantime-serde = "1.1"
 itertools = "0.10"
 lazy_static = "1.4"
-meter-core = { git = "https://github.com/GreptimeTeam/greptime-meter.git", rev = "80b72716dcde47ec4161478416a5c6c21343364d" }
+meter-core = { git = "https://github.com/GreptimeTeam/greptime-meter.git", rev = "abbd357c1e193cd270ea65ee7652334a150b628f" }
 mockall = "0.11.4"
 moka = "0.12"
 num_cpus = "1.16"
@@ -134,7 +127,7 @@ reqwest = { version = "0.11", default-features = false, features = [
 rskafka = "0.5"
 rust_decimal = "1.33"
 serde = { version = "1.0", features = ["derive"] }
-serde_json = { version = "1.0", features = ["float_roundtrip"] }
+serde_json = "1.0"
 serde_with = "3"
 smallvec = { version = "1", features = ["serde"] }
 snafu = "0.7"
@@ -171,7 +164,6 @@ common-grpc-expr = { path = "src/common/grpc-expr" }
 common-macro = { path = "src/common/macro" }
 common-mem-prof = { path = "src/common/mem-prof" }
 common-meta = { path = "src/common/meta" }
-common-plugins = { path = "src/common/plugins" }
 common-procedure = { path = "src/common/procedure" }
 common-procedure-test = { path = "src/common/procedure-test" }
 common-query = { path = "src/common/query" }
@@ -209,7 +201,7 @@ table = { path = "src/table" }
 
 [workspace.dependencies.meter-macros]
 git = "https://github.com/GreptimeTeam/greptime-meter.git"
-rev = "80b72716dcde47ec4161478416a5c6c21343364d"
+rev = "abbd357c1e193cd270ea65ee7652334a150b628f"
 
 [profile.release]
 debug = 1

Makefile

@@ -3,7 +3,6 @@ CARGO_PROFILE ?=
 FEATURES ?=
 TARGET_DIR ?=
 TARGET ?=
-BUILD_BIN ?= greptime
 CARGO_BUILD_OPTS := --locked
 IMAGE_REGISTRY ?= docker.io
 IMAGE_NAMESPACE ?= greptime
@@ -46,10 +45,6 @@ ifneq ($(strip $(TARGET)),)
 	CARGO_BUILD_OPTS += --target ${TARGET}
 endif
 
-ifneq ($(strip $(BUILD_BIN)),)
-	CARGO_BUILD_OPTS += --bin ${BUILD_BIN}
-endif
-
 ifneq ($(strip $(RELEASE)),)
 	CARGO_BUILD_OPTS += --release
 endif

README.md

@@ -6,154 +6,145 @@
   </picture>
 </p>
 
-<h1 align="center">Cloud-scale, Fast and Efficient Time Series Database</h1>
 
-<div align="center">
 <h3 align="center">
-  <a href="https://greptime.com/product/cloud">GreptimeCloud</a> |
-  <a href="https://docs.greptime.com/">User guide</a> |
-  <a href="https://greptimedb.rs/">API Docs</a> |
-  <a href="https://github.com/GreptimeTeam/greptimedb/issues/3412">Roadmap 2024</a>
-</h4>
+    The next-generation hybrid time-series/analytics processing database in the cloud
+</h3>
 
-<a href="https://github.com/GreptimeTeam/greptimedb/releases/latest">
-<img src="https://img.shields.io/github/v/release/GreptimeTeam/greptimedb.svg" alt="Version"/>
-</a>
-<a href="https://github.com/GreptimeTeam/greptimedb/releases/latest">
-<img src="https://img.shields.io/github/release-date/GreptimeTeam/greptimedb.svg" alt="Releases"/>
-</a>
-<a href="https://hub.docker.com/r/greptime/greptimedb/">
-<img src="https://img.shields.io/docker/pulls/greptime/greptimedb.svg" alt="Docker Pulls"/>
-</a>
-<a href="https://github.com/GreptimeTeam/greptimedb/actions/workflows/develop.yml">
-<img src="https://github.com/GreptimeTeam/greptimedb/actions/workflows/develop.yml/badge.svg" alt="GitHub Actions"/>
-</a>
-<a href="https://codecov.io/gh/GrepTimeTeam/greptimedb">
-<img src="https://codecov.io/gh/GrepTimeTeam/greptimedb/branch/main/graph/badge.svg?token=FITFDI3J3C" alt="Codecov"/>
-</a>
-<a href="https://github.com/greptimeTeam/greptimedb/blob/main/LICENSE">
-<img src="https://img.shields.io/github/license/greptimeTeam/greptimedb" alt="License"/>
-</a>
+<p align="center">
+    <a href="https://codecov.io/gh/GrepTimeTeam/greptimedb"><img src="https://codecov.io/gh/GrepTimeTeam/greptimedb/branch/main/graph/badge.svg?token=FITFDI3J3C"></img></a>
+    &nbsp;
+    <a href="https://github.com/GreptimeTeam/greptimedb/actions/workflows/develop.yml"><img src="https://github.com/GreptimeTeam/greptimedb/actions/workflows/develop.yml/badge.svg" alt="CI"></img></a>
+    &nbsp;
+    <a href="https://github.com/greptimeTeam/greptimedb/blob/main/LICENSE"><img src="https://img.shields.io/github/license/greptimeTeam/greptimedb"></a>
+</p>
 
-<br/>
+<p align="center">
+    <a href="https://twitter.com/greptime"><img src="https://img.shields.io/badge/twitter-follow_us-1d9bf0.svg"></a>
+    &nbsp;
+    <a href="https://www.linkedin.com/company/greptime/"><img src="https://img.shields.io/badge/linkedin-connect_with_us-0a66c2.svg"></a>
+    &nbsp;
+    <a href="https://greptime.com/slack"><img src="https://img.shields.io/badge/slack-GreptimeDB-0abd59?logo=slack" alt="slack" /></a>
+</p>
 
-<a href="https://greptime.com/slack">
-<img src="https://img.shields.io/badge/slack-GreptimeDB-0abd59?logo=slack&style=for-the-badge" alt="Slack"/>
-</a>
-<a href="https://twitter.com/greptime">
-<img src="https://img.shields.io/badge/twitter-follow_us-1d9bf0.svg?style=for-the-badge" alt="Twitter"/>
-</a>
-<a href="https://www.linkedin.com/company/greptime/">
-<img src="https://img.shields.io/badge/linkedin-connect_with_us-0a66c2.svg?style=for-the-badge" alt="LinkedIn"/>
-</a>
-</div>
+## What is GreptimeDB
 
-## Introduction
+GreptimeDB is an open-source time-series database focusing on efficiency, scalability, and analytical capabilities.
+It's designed to work on infrastructure of the cloud era, and users benefit from its elasticity and commodity storage.
 
-**GreptimeDB** is an open-source time-series database focusing on efficiency, scalability, and analytical capabilities.
-Designed to work on infrastructure of the cloud era, GreptimeDB benefits users with its elasticity and commodity storage, offering a fast and cost-effective **alternative to InfluxDB** and a **long-term storage for Prometheus**.
+Our core developers have been building time-series data platforms for years. Based on their best-practices, GreptimeDB is born to give you:
 
-## Why GreptimeDB
+- Optimized columnar layout for handling time-series data; compacted, compressed, and stored on various storage backends, particularly cloud object storage with 50x cost efficiency.
+- Fully open-source distributed cluster architecture that harnesses the power of cloud-native elastic computing resources.
+- Seamless scalability from a standalone binary at edge to a robust, highly available distributed cluster in cloud, with a transparent experience for both developers and administrators.
+- Native SQL and PromQL for queries, and Python scripting to facilitate complex analytical tasks.
+- Flexible indexing capabilities and distributed, parallel-processing query engine, tackling high cardinality issues down.
+- Widely adopted database protocols and APIs, including MySQL, PostgreSQL, and Prometheus Remote Storage, etc.
 
-Our core developers have been building time-series data platforms for years. Based on our best-practices, GreptimeDB is born to give you:
+## Quick Start
 
-* **Easy horizontal scaling**
-
-  Seamless scalability from a standalone binary at edge to a robust, highly available distributed cluster in cloud, with a transparent experience for both developers and administrators.
-
-* **Analyzing time-series data**
-
-  Query your time-series data with SQL and PromQL. Use Python scripts to facilitate complex analytical tasks.
-
-* **Cloud-native distributed database**
-
-  Fully open-source distributed cluster architecture that harnesses the power of cloud-native elastic computing resources.
-
-* **Performance and Cost-effective**
-
-  Flexible indexing capabilities and distributed, parallel-processing query engine, tackling high cardinality issues down. Optimized columnar layout for handling time-series data; compacted, compressed, and stored on various storage backends, particularly cloud object storage with 50x cost efficiency.
-
-* **Compatible with InfluxDB, Prometheus and more protocols**
-
-  Widely adopted database protocols and APIs, including MySQL, PostgreSQL, and Prometheus Remote Storage, etc. [Read more](https://docs.greptime.com/user-guide/clients/overview).
-
-## Try GreptimeDB
-
-### 1. [GreptimePlay](https://greptime.com/playground)
+### [GreptimePlay](https://greptime.com/playground)
 
 Try out the features of GreptimeDB right from your browser.
 
-### 2. [GreptimeCloud](https://console.greptime.cloud/)
+### Build
 
-Start instantly with a free cluster.
+#### Build from Source
 
-### 3. Docker Image
+To compile GreptimeDB from source, you'll need:
 
-To install GreptimeDB locally, the recommended way is via Docker:
+- C/C++ Toolchain: provides basic tools for compiling and linking. This is
+  available as `build-essential` on ubuntu and similar name on other platforms.
+- Rust: the easiest way to install Rust is to use
+  [`rustup`](https://rustup.rs/), which will check our `rust-toolchain` file and
+  install correct Rust version for you.
+- Protobuf: `protoc` is required for compiling `.proto` files. `protobuf` is
+  available from major package manager on macos and linux distributions. You can
+  find an installation instructions [here](https://grpc.io/docs/protoc-installation/).
+  **Note that `protoc` version needs to be >= 3.15** because we have used the `optional`
+  keyword. You can check it with `protoc --version`.
+- python3-dev or python3-devel(Optional feature, only needed if you want to run scripts
+  in CPython, and also need to enable `pyo3_backend` feature when compiling(by `cargo run -F pyo3_backend` or add `pyo3_backend` to src/script/Cargo.toml 's `features.default` like `default = ["python", "pyo3_backend]`)): this install a Python shared library required for running Python
+  scripting engine(In CPython Mode). This is available as `python3-dev` on
+  ubuntu, you can install it with `sudo apt install python3-dev`, or
+  `python3-devel` on RPM based distributions (e.g. Fedora, Red Hat, SuSE). Mac's
+  `Python3` package should have this shared library by default. More detail for compiling with PyO3 can be found in [PyO3](https://pyo3.rs/v0.18.1/building_and_distribution#configuring-the-python-version)'s documentation.
 
-```shell
-docker pull greptime/greptimedb
+#### Build with Docker
+
+A docker image with necessary dependencies is provided:
+
+```
+docker build --network host -f docker/Dockerfile -t greptimedb .
 ```
 
-Start a GreptimeDB container with:
+### Run
+
+Start GreptimeDB from source code, in standalone mode:
 
-```shell
-docker run --rm --name greptime --net=host greptime/greptimedb standalone start
 ```
-
-Read more about [Installation](https://docs.greptime.com/getting-started/installation/overview) on docs.
-
-## Getting Started
-
-* [Quickstart](https://docs.greptime.com/getting-started/quick-start/overview)
-* [Write Data](https://docs.greptime.com/user-guide/clients/overview)
-* [Query Data](https://docs.greptime.com/user-guide/query-data/overview)
-* [Operations](https://docs.greptime.com/user-guide/operations/overview)
-
-## Build
-
-Check the prerequisite:
-
-* [Rust toolchain](https://www.rust-lang.org/tools/install) (nightly)
-* [Protobuf compiler](https://grpc.io/docs/protoc-installation/) (>= 3.15)
-* Python toolchain (optional): Required only if built with PyO3 backend. More detail for compiling with PyO3 can be found in its [documentation](https://pyo3.rs/v0.18.1/building_and_distribution#configuring-the-python-version).
-
-Build GreptimeDB binary:
-
-```shell
-make
-```
-
-Run a standalone server:
-
-```shell
 cargo run -- standalone start
 ```
 
-## Extension
+Or if you built from docker:
+
+```
+docker run -p 4002:4002 -v "$(pwd):/tmp/greptimedb" greptime/greptimedb standalone start
+```
+
+Please see the online document site for more installation options and [operations info](https://docs.greptime.com/user-guide/operations/overview).
+
+### Get started
+
+Read the [complete getting started guide](https://docs.greptime.com/getting-started/overview) on our [official document site](https://docs.greptime.com/).
+
+To write and query data, GreptimeDB is compatible with multiple [protocols and clients](https://docs.greptime.com/user-guide/clients/overview).
+
+## Resources
+
+### Installation
+
+- [Pre-built Binaries](https://greptime.com/download):
+  For Linux and macOS, you can easily download pre-built binaries including official releases and nightly builds that are ready to use.
+  In most cases, downloading the version without PyO3 is sufficient. However, if you plan to run scripts in CPython (and use Python packages like NumPy and Pandas), you will need to download the version with PyO3 and install a Python with the same version as the Python in the PyO3 version.
+  We recommend using virtualenv for the installation process to manage multiple Python versions.
+- [Docker Images](https://hub.docker.com/r/greptime/greptimedb)(**recommended**): pre-built
+  Docker images, this is the easiest way to try GreptimeDB. By default it runs CPython script with `pyo3_backend` enabled.
+- [`gtctl`](https://github.com/GreptimeTeam/gtctl): the command-line tool for
+  Kubernetes deployment
+
+### Documentation
+
+- GreptimeDB [User Guide](https://docs.greptime.com/user-guide/concepts/overview)
+- GreptimeDB [Developer
+  Guide](https://docs.greptime.com/developer-guide/overview.html)
+- GreptimeDB [internal code document](https://greptimedb.rs)
 
 ### Dashboard
-
 - [The dashboard UI for GreptimeDB](https://github.com/GreptimeTeam/dashboard)
 
 ### SDK
 
+- [GreptimeDB C++ Client](https://github.com/GreptimeTeam/greptimedb-client-cpp)
+- [GreptimeDB Erlang Client](https://github.com/GreptimeTeam/greptimedb-client-erl)
 - [GreptimeDB Go Ingester](https://github.com/GreptimeTeam/greptimedb-ingester-go)
 - [GreptimeDB Java Ingester](https://github.com/GreptimeTeam/greptimedb-ingester-java)
-- [GreptimeDB C++ Ingester](https://github.com/GreptimeTeam/greptimedb-ingester-cpp)
-- [GreptimeDB Erlang Ingester](https://github.com/GreptimeTeam/greptimedb-ingester-erl)
-- [GreptimeDB Rust Ingester](https://github.com/GreptimeTeam/greptimedb-ingester-rust)
-- [GreptimeDB JavaScript Ingester](https://github.com/GreptimeTeam/greptime-ingester-js)
+- [GreptimeDB Python Client](https://github.com/GreptimeTeam/greptimedb-client-py) (WIP)
+- [GreptimeDB Rust Client](https://github.com/GreptimeTeam/greptimedb-client-rust)
+- [GreptimeDB JavaScript Client](https://github.com/GreptimeTeam/greptime-js-sdk)
 
 ### Grafana Dashboard
 
-Our official Grafana dashboard is available at [grafana](grafana/README.md) directory.
+Our official Grafana dashboard is available at [grafana](./grafana/README.md) directory.
 
 ## Project Status
 
-The current version has not yet reached General Availability version standards.
-In line with our Greptime 2024 Roadmap, we plan to achieve a production-level
-version with the update to v1.0 in August. [[Join Force]](https://github.com/GreptimeTeam/greptimedb/issues/3412)
+This project is in its early stage and under heavy development. We move fast and
+break things. Benchmark on development branch may not represent its potential
+performance. We release pre-built binaries constantly for functional
+evaluation. Do not use it in production at the moment.
+
+For future plans, check out [GreptimeDB roadmap](https://github.com/GreptimeTeam/greptimedb/issues/669).
 
 ## Community
 
@@ -163,12 +154,12 @@ and what went wrong. If you have any questions or if you would like to get invol
 community, please check out:
 
 - GreptimeDB Community on [Slack](https://greptime.com/slack)
-- GreptimeDB [GitHub Discussions forum](https://github.com/GreptimeTeam/greptimedb/discussions)
-- Greptime official [website](https://greptime.com)
+- GreptimeDB GitHub [Discussions](https://github.com/GreptimeTeam/greptimedb/discussions)
+- Greptime official [Website](https://greptime.com)
 
 In addition, you may:
 
-- View our official [Blog](https://greptime.com/blogs/)
+- View our official [Blog](https://greptime.com/blogs/index)
 - Connect us with [Linkedin](https://www.linkedin.com/company/greptime/)
 - Follow us on [Twitter](https://twitter.com/greptime)
 
@@ -179,7 +170,7 @@ open contributions and allowing you to use the software however you want.
 
 ## Contributing
 
-Please refer to [contribution guidelines](CONTRIBUTING.md) and [internal concepts docs](https://docs.greptime.com/contributor-guide/overview.html) for more information.
+Please refer to [contribution guidelines](CONTRIBUTING.md) for more information.
 
 ## Acknowledgement
 

benchmarks/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 arrow.workspace = true
 chrono.workspace = true

benchmarks/src/bin/nyc-taxi.rs

@@ -29,7 +29,7 @@ use client::api::v1::column::Values;
 use client::api::v1::{
     Column, ColumnDataType, ColumnDef, CreateTableExpr, InsertRequest, InsertRequests, SemanticType,
 };
-use client::{Client, Database, OutputData, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
+use client::{Client, Database, Output, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
 use futures_util::TryStreamExt;
 use indicatif::{MultiProgress, ProgressBar, ProgressStyle};
 use parquet::arrow::arrow_reader::ParquetRecordBatchReaderBuilder;
@@ -502,9 +502,9 @@ async fn do_query(num_iter: usize, db: &Database, table_name: &str) {
         for i in 0..num_iter {
             let now = Instant::now();
             let res = db.sql(&query).await.unwrap();
-            match res.data {
-                OutputData::AffectedRows(_) | OutputData::RecordBatches(_) => (),
-                OutputData::Stream(stream) => {
+            match res {
+                Output::AffectedRows(_) | Output::RecordBatches(_) => (),
+                Output::Stream(stream) => {
                     stream.try_collect::<Vec<_>>().await.unwrap();
                 }
             }

codecov.yml

@@ -8,6 +8,5 @@ coverage:
 ignore:
   - "**/error*.rs" # ignore all error.rs files
   - "tests/runner/*.rs" # ignore integration test runner
-  - "tests-integration/**/*.rs" # ignore integration tests
 comment:                  # this is a top-level key
   layout: "diff"

config/datanode.example.toml

@@ -134,22 +134,10 @@ create_on_compaction = "auto"
 apply_on_query = "auto"
 # Memory threshold for performing an external sort during index creation.
 # Setting to empty will disable external sorting, forcing all sorting operations to happen in memory.
-mem_threshold_on_create = "64M"
+mem_threshold_on_create = "64MB"
 # File system path to store intermediate files for external sorting (default `{data_home}/index_intermediate`).
 intermediate_path = ""
 
-[region_engine.mito.memtable]
-# Memtable type.
-# - "partition_tree": partition tree memtable
-# - "time_series": time-series memtable (deprecated)
-type = "partition_tree"
-# The max number of keys in one shard.
-index_max_keys_per_shard = 8192
-# The max rows of data inside the actively writing buffer in one shard.
-data_freeze_threshold = 32768
-# Max dictionary bytes.
-fork_dictionary_bytes = "1GiB"
-
 # Log options, see `standalone.example.toml`
 # [logging]
 # dir = "/tmp/greptimedb/logs"

config/frontend.example.toml

@@ -31,7 +31,6 @@ runtime_size = 2
 mode = "disable"
 cert_path = ""
 key_path = ""
-watch = false
 
 # PostgresSQL server options, see `standalone.example.toml`.
 [postgres]
@@ -44,7 +43,6 @@ runtime_size = 2
 mode = "disable"
 cert_path = ""
 key_path = ""
-watch = false
 
 # OpenTSDB protocol options, see `standalone.example.toml`.
 [opentsdb]

config/standalone.example.toml

@@ -44,8 +44,6 @@ mode = "disable"
 cert_path = ""
 # Private key file path.
 key_path = ""
-# Watch for Certificate and key file change and auto reload
-watch = false
 
 # PostgresSQL server options.
 [postgres]
@@ -64,8 +62,6 @@ mode = "disable"
 cert_path = ""
 # private key file path.
 key_path = ""
-# Watch for Certificate and key file change and auto reload
-watch = false
 
 # OpenTSDB protocol options.
 [opentsdb]
@@ -122,7 +118,7 @@ sync_period = "1000ms"
 # Number of topics to be created upon start.
 # num_topics = 64
 # Topic selector type.
-# Available selector types:
+# Available selector types: 
 # - "round_robin" (default)
 # selector_type = "round_robin"
 # The prefix of topic name.
@@ -244,18 +240,6 @@ mem_threshold_on_create = "64M"
 # File system path to store intermediate files for external sorting (default `{data_home}/index_intermediate`).
 intermediate_path = ""
 
-[region_engine.mito.memtable]
-# Memtable type.
-# - "partition_tree": partition tree memtable
-# - "time_series": time-series memtable (deprecated)
-type = "partition_tree"
-# The max number of keys in one shard.
-index_max_keys_per_shard = 8192
-# The max rows of data inside the actively writing buffer in one shard.
-data_freeze_threshold = 32768
-# Max dictionary bytes.
-fork_dictionary_bytes = "1GiB"
-
 # Log options
 # [logging]
 # Specify logs directory.
@@ -266,11 +250,10 @@ fork_dictionary_bytes = "1GiB"
 # enable_otlp_tracing = false
 # tracing exporter endpoint with format `ip:port`, we use grpc oltp as exporter, default endpoint is `localhost:4317`
 # otlp_endpoint = "localhost:4317"
+# The percentage of tracing will be sampled and exported. Valid range `[0, 1]`, 1 means all traces are sampled, 0 means all traces are not sampled, the default value is 1. ratio > 1 are treated as 1. Fractions < 0 are treated as 0
+# tracing_sample_ratio = 1.0
 # Whether to append logs to stdout. Defaults to true.
 # append_stdout = true
-# The percentage of tracing will be sampled and exported. Valid range `[0, 1]`, 1 means all traces are sampled, 0 means all traces are not sampled, the default value is 1. ratio > 1 are treated as 1. Fractions < 0 are treated as 0
-# [logging.tracing_sample_ratio]
-# default_ratio = 0.0
 
 # Standalone export the metrics generated by itself
 # encoded to Prometheus remote-write format

docs/benchmarks/tsbs/v0.7.0.md

@@ -1,50 +0,0 @@
-# TSBS benchmark - v0.7.0
-
-## Environment
-
-### Local
-|        |                                    |
-| ------ | ---------------------------------- |
-| CPU    | AMD Ryzen 7 7735HS (8 core 3.2GHz) |
-| Memory | 32GB                               |
-| Disk   | SOLIDIGM SSDPFKNU010TZ             |
-| OS     | Ubuntu 22.04.2 LTS                 |
-
-### Amazon EC2
-
-|         |                |
-| ------- | -------------- |
-| Machine | c5d.2xlarge    |
-| CPU     | 8 core         |
-| Memory  | 16GB           |
-| Disk    | 50GB (GP3)     |
-| OS      | Ubuntu 22.04.1 |
-
-
-## Write performance
-
-| Environment        | Ingest rate (rows/s)  |
-| ------------------ | --------------------- |
-| Local              | 3695814.64            |
-| EC2 c5d.2xlarge    | 2987166.64            |
-
-
-## Query performance
-
-| Query type            | Local (ms) | EC2 c5d.2xlarge (ms)   |
-| --------------------- | ---------- | ---------------------- |
-| cpu-max-all-1         | 30.56      | 54.74                  |
-| cpu-max-all-8         | 52.69      | 70.50                  |
-| double-groupby-1      | 664.30     | 1366.63                |
-| double-groupby-5      | 1391.26    | 2141.71                |
-| double-groupby-all    | 2828.94    | 3389.59                |
-| groupby-orderby-limit | 718.92     | 1213.90                |
-| high-cpu-1            | 29.21      | 52.98                  |
-| high-cpu-all          | 5514.12    | 7194.91                |
-| lastpoint             | 7571.40    | 9423.41                |
-| single-groupby-1-1-1  | 19.09      | 7.77                   |
-| single-groupby-1-1-12 | 27.28      | 51.64                  |
-| single-groupby-1-8-1  | 31.85      | 11.64                  |
-| single-groupby-5-1-1  | 16.14      | 9.67                   |
-| single-groupby-5-1-12 | 27.21      | 53.62                  |
-| single-groupby-5-8-1  | 39.62      | 14.96                  |

docs/rfcs/2023-05-09-distributed-planner.md

@@ -79,7 +79,7 @@ This RFC proposes to add a new expression node `MergeScan` to merge result from
 │               │    │                             │
 └─Frontend──────┘    └─Remote-Sources──────────────┘
 ```
-This merge operation simply chains all the underlying remote data sources and return `RecordBatch`, just like a coalesce op. And each remote sources is a gRPC query to datanode via the substrait logical plan interface. The plan is transformed and divided from the original query that comes to frontend.
+This merge operation simply chains all the the underlying remote data sources and return `RecordBatch`, just like a coalesce op. And each remote sources is a gRPC query to datanode via the substrait logical plan interface. The plan is transformed and divided from the original query that comes to frontend.
 
 ## Commutativity of MergeScan
 

docs/rfcs/2024-02-21-multi-dimension-partition-rule/rfc.md

@@ -1,101 +0,0 @@
----
-Feature Name:  Multi-dimension Partition Rule
-Tracking Issue: https://github.com/GreptimeTeam/greptimedb/issues/3351
-Date: 2024-02-21
-Author: "Ruihang Xia <waynestxia@gmail.com>"
----
-
-# Summary
-
-A new region partition scheme that runs on multiple dimensions of the key space. The partition rule is defined by a set of simple expressions on the partition key columns.
-
-# Motivation
-
-The current partition rule is from MySQL's [`RANGE Partition`](https://dev.mysql.com/doc/refman/8.0/en/partitioning-range.html), which is based on a single dimension. It is sort of a [Hilbert Curve](https://en.wikipedia.org/wiki/Hilbert_curve) and pick several point on the curve to divide the space. It is neither easy to understand how the data get partitioned nor flexible enough to handle complex partitioning requirements.
-
-Considering the future requirements like region repartitioning or autonomous rebalancing, where both workload and partition may change frequently. Here proposes a new region partition scheme that uses a set of simple expressions on the partition key columns to divide the key space.
-
-# Details
-
-## Partition rule
-
-First, we define a simple expression that can be used to define the partition rule. The simple expression is a binary expression expression on the partition key columns that can be evaluated to a boolean value. The binary operator is limited to comparison operators only, like `=`, `!=`, `>`, `>=`, `<`, `<=`. And the operands are limited either literal value or partition column.
-
-Example of valid simple expressions are $`col_A = 10`$, $`col_A \gt 10 \& col_B \gt 20`$ or $`col_A \ne 10`$.
-
-Those expressions can be used as predicates to divide the key space into different regions. The following example have two partition columns `Col A` and `Col B`, and four partitioned regions.
-
-```math
-\left\{\begin{aligned}
- 
-&col_A \le 10 &Region_1 \\
-&10 \lt col_A \& col_A \le 20 &Region_2 \\
-&20 \lt col_A \space \& \space col_B \lt 100 &Region_3 \\
-&20 \lt col_A \space \& \space col_B \ge 100 &Region_4
-
-\end{aligned}\right\}
-```
-
-An advantage of this scheme is that it is easy to understand how the data get partitioned. The above example can be visualized in a 2D space (two partition column is involved in the example).
-
-![example](2d-example.png)
-
-Here each expression draws a line in the 2D space. Managing data partitioning becomes a matter of drawing lines in the key space.
-
-To make it easy to use, there is a "default region" which catches all the data that doesn't match any of previous expressions. The default region exist by default and do not need to specify. It is also possible to remove this default region if the DB finds it is not necessary.
-
-## SQL interface
-
-The SQL interface is in response to two parts: specifying the partition columns and the partition rule. Thouth we are targeting an autonomous system, it's still allowed to give some bootstrap rules or hints on creating table.
-
-Partition column is specified by `PARTITION ON COLUMNS` sub-clause in `CREATE TABLE`:
-
-```sql
-CREATE TABLE t (...)
-PARTITION ON COLUMNS (...) ();
-```
-
-Two following brackets are for partition columns and partition rule respectively.
-
-Columns provided here are only used as an allow-list of how the partition rule can be defined. Which means (a) the sequence between columns doesn't matter, (b) the columns provided here are not necessarily being used in the partition rule.
-
-The partition rule part is a list of comma-separated simple expressions. Expressions here are not corresponding to region, as they might be changed by system to fit various workload.
-
-A full example of `CREATE TABLE` with partition rule is:
-
-```sql
-CREATE TABLE IF NOT EXISTS demo (
-  a STRING,
-  b STRING,
-  c STRING,
-  d STRING,
-  ts TIMESTAMP,
-  memory DOUBLE,
-  TIME INDEX (ts),
-  PRIMARY KEY (a, b, c, d)
-)
-PARTITION ON COLUMNS (c, b, a) (
-  a < 10,
-  10 >= a AND a < 20,
-  20 >= a AND b < 100,
-  20 >= a AND b > 100
-)
-```
-
-## Combine with storage
-
-Examining columns separately suits our columnar storage very well in two aspects.
-
-1. The simple expression can be pushed down to storage and file format, and is likely to hit existing index. Makes pruning operation very efficient.
-
-2. Columns in columnar storage are not tightly coupled like in the traditional row storages, which means we can easily add or remove columns from partition rule without much impact (like a global reshuffle) on data.
-
-The data file itself can be "projected" to the key space as a polyhedron, it is guaranteed that each plane is in parallel with some coordinate planes (in a 2D scenario, this is saying that all the files can be projected to a rectangle). Thus partition or repartition also only need to consider related columns.
-
-![sst-project](sst-project.png)
-
-An additional limitation is that considering how the index works and how we organize the primary keys at present, the partition columns are limited to be a subset of primary keys for better performance.
-
-# Drawbacks
-
-This is a breaking change.

grafana/greptimedb.json

@@ -66,7 +66,7 @@
   },
   "editable": true,
   "fiscalYearStartMonth": 0,
-  "graphTooltip": 1,
+  "graphTooltip": 0,
   "id": null,
   "links": [],
   "liveNow": false,
@@ -2116,7 +2116,7 @@
               }
             ]
           },
-          "unit": "none"
+          "unit": "bytes"
         },
         "overrides": []
       },
@@ -2126,7 +2126,7 @@
         "x": 0,
         "y": 61
       },
-      "id": 17,
+      "id": 12,
       "interval": "1s",
       "options": {
         "legend": {
@@ -2147,8 +2147,8 @@
             "uid": "${DS_PROMETHEUS-1}"
           },
           "disableTextWrap": false,
-          "editorMode": "code",
-          "expr": "rate(raft_engine_sync_log_duration_seconds_count[2s])",
+          "editorMode": "builder",
+          "expr": "histogram_quantile(0.95, sum by(le) (rate(raft_engine_write_size_bucket[$__rate_interval])))",
           "fullMetaSearch": false,
           "includeNullMetadata": false,
           "instant": false,
@@ -2158,7 +2158,7 @@
           "useBackend": false
         }
       ],
-      "title": "raft engine sync count",
+      "title": "wal write size",
       "type": "timeseries"
     },
     {
@@ -2378,120 +2378,6 @@
       ],
       "title": "raft engine write duration seconds",
       "type": "timeseries"
-    },
-    {
-      "datasource": {
-        "type": "prometheus",
-        "uid": "${DS_PROMETHEUS-1}"
-      },
-      "fieldConfig": {
-        "defaults": {
-          "color": {
-            "mode": "palette-classic"
-          },
-          "custom": {
-            "axisBorderShow": false,
-            "axisCenteredZero": false,
-            "axisColorMode": "text",
-            "axisLabel": "",
-            "axisPlacement": "auto",
-            "barAlignment": 0,
-            "drawStyle": "line",
-            "fillOpacity": 0,
-            "gradientMode": "none",
-            "hideFrom": {
-              "legend": false,
-              "tooltip": false,
-              "viz": false
-            },
-            "insertNulls": false,
-            "lineInterpolation": "linear",
-            "lineWidth": 1,
-            "pointSize": 5,
-            "scaleDistribution": {
-              "type": "linear"
-            },
-            "showPoints": "auto",
-            "spanNulls": false,
-            "stacking": {
-              "group": "A",
-              "mode": "none"
-            },
-            "thresholdsStyle": {
-              "mode": "off"
-            }
-          },
-          "mappings": [],
-          "thresholds": {
-            "mode": "absolute",
-            "steps": [
-              {
-                "color": "green",
-                "value": null
-              },
-              {
-                "color": "red",
-                "value": 80
-              }
-            ]
-          },
-          "unit": "bytes"
-        },
-        "overrides": []
-      },
-      "gridPos": {
-        "h": 7,
-        "w": 12,
-        "x": 12,
-        "y": 68
-      },
-      "id": 12,
-      "interval": "1s",
-      "options": {
-        "legend": {
-          "calcs": [],
-          "displayMode": "list",
-          "placement": "bottom",
-          "showLegend": true
-        },
-        "tooltip": {
-          "mode": "single",
-          "sort": "none"
-        }
-      },
-      "targets": [
-        {
-          "datasource": {
-            "type": "prometheus",
-            "uid": "${DS_PROMETHEUS-1}"
-          },
-          "disableTextWrap": false,
-          "editorMode": "code",
-          "expr": "histogram_quantile(0.95, sum by(le) (rate(raft_engine_write_size_bucket[$__rate_interval])))",
-          "fullMetaSearch": false,
-          "includeNullMetadata": false,
-          "instant": false,
-          "legendFormat": "req-size-p95",
-          "range": true,
-          "refId": "A",
-          "useBackend": false
-        },
-        {
-          "datasource": {
-            "type": "prometheus",
-            "uid": "${DS_PROMETHEUS-1}"
-          },
-          "editorMode": "code",
-          "expr": "rate(raft_engine_write_size_sum[$__rate_interval])",
-          "hide": false,
-          "instant": false,
-          "legendFormat": "throughput",
-          "range": true,
-          "refId": "B"
-        }
-      ],
-      "title": "wal write size",
-      "type": "timeseries"
     }
   ],
   "refresh": "10s",
@@ -2501,13 +2387,13 @@
     "list": []
   },
   "time": {
-    "from": "now-30m",
+    "from": "now-3h",
     "to": "now"
   },
   "timepicker": {},
   "timezone": "",
   "title": "GreptimeDB",
   "uid": "e7097237-669b-4f8d-b751-13067afbfb68",
-  "version": 12,
+  "version": 9,
   "weekStart": ""
 }

scripts/fetch-dashboard-assets.sh

@@ -1,7 +1,8 @@
 #!/usr/bin/env bash
 
 # This script is used to download built dashboard assets from the "GreptimeTeam/dashboard" repository.
-set -ex
+
+set -e -x
 
 declare -r SCRIPT_DIR=$(cd $(dirname ${0}) >/dev/null 2>&1 && pwd)
 declare -r ROOT_DIR=$(dirname ${SCRIPT_DIR})
@@ -12,34 +13,13 @@ RELEASE_VERSION="$(cat $STATIC_DIR/VERSION | tr -d '\t\r\n ')"
 
 echo "Downloading assets to dir: $OUT_DIR"
 cd $OUT_DIR
-
-if [[ -z "$GITHUB_PROXY_URL" ]]; then
-  GITHUB_URL="https://github.com"
-else
-  GITHUB_URL="${GITHUB_PROXY_URL%/}"
-fi
-
-function retry_fetch() {
-    local url=$1
-    local filename=$2
-
-    curl --connect-timeout 10 --retry 3 -fsSL $url --output $filename || {
-        echo "Failed to download $url"
-        echo "You may try to set http_proxy and https_proxy environment variables."
-        if [[ -z "$GITHUB_PROXY_URL" ]]; then
-          echo "You may try to set GITHUB_PROXY_URL=http://mirror.ghproxy.com/https://github.com/"
-        fi
-        exit 1
-     }
-}
-
 # Download the SHA256 checksum attached to the release. To verify the integrity
 # of the download, this checksum will be used to check the download tar file
 # containing the built dashboard assets.
-retry_fetch "${GITHUB_URL}/GreptimeTeam/dashboard/releases/download/${RELEASE_VERSION}/sha256.txt" sha256.txt
+curl -Ls https://github.com/GreptimeTeam/dashboard/releases/download/$RELEASE_VERSION/sha256.txt --output sha256.txt
 
 # Download the tar file containing the built dashboard assets.
-retry_fetch "${GITHUB_URL}/GreptimeTeam/dashboard/releases/download/${RELEASE_VERSION}/build.tar.gz" build.tar.gz
+curl -L https://github.com/GreptimeTeam/dashboard/releases/download/$RELEASE_VERSION/build.tar.gz --output build.tar.gz
 
 # Verify the checksums match; exit if they don't.
 case "$(uname -s)" in

src/api/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 common-base.workspace = true
 common-decimal.workspace = true

src/api/src/helper.rs

@@ -707,6 +707,7 @@ pub fn pb_values_to_vector_ref(data_type: &ConcreteDataType, values: Values) ->
 }
 
 pub fn pb_values_to_values(data_type: &ConcreteDataType, values: Values) -> Vec<Value> {
+    // TODO(fys): use macros to optimize code
     match data_type {
         ConcreteDataType::Int64(_) => values
             .i64_values

src/auth/Cargo.toml

@@ -8,9 +8,6 @@ license.workspace = true
 default = []
 testing = []
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 async-trait.workspace = true

src/auth/src/common.rs

@@ -40,7 +40,7 @@ pub fn user_provider_from_option(opt: &String) -> Result<UserProviderRef> {
     match name {
         STATIC_USER_PROVIDER => {
             let provider =
-                StaticUserProvider::new(content).map(|p| Arc::new(p) as UserProviderRef)?;
+                StaticUserProvider::try_from(content).map(|p| Arc::new(p) as UserProviderRef)?;
             Ok(provider)
         }
         _ => InvalidConfigSnafu {

src/auth/src/user_provider/static_user_provider.rs

@@ -23,7 +23,7 @@ use secrecy::ExposeSecret;
 use snafu::{ensure, OptionExt, ResultExt};
 
 use crate::error::{
-    IllegalParamSnafu, InvalidConfigSnafu, IoSnafu, Result, UnsupportedPasswordTypeSnafu,
+    Error, IllegalParamSnafu, InvalidConfigSnafu, IoSnafu, Result, UnsupportedPasswordTypeSnafu,
     UserNotFoundSnafu, UserPasswordMismatchSnafu,
 };
 use crate::user_info::DefaultUserInfo;
@@ -31,12 +31,10 @@ use crate::{auth_mysql, Identity, Password, UserInfoRef, UserProvider};
 
 pub(crate) const STATIC_USER_PROVIDER: &str = "static_user_provider";
 
-pub(crate) struct StaticUserProvider {
-    users: HashMap<String, Vec<u8>>,
-}
+impl TryFrom<&str> for StaticUserProvider {
+    type Error = Error;
 
-impl StaticUserProvider {
-    pub(crate) fn new(value: &str) -> Result<Self> {
+    fn try_from(value: &str) -> Result<Self> {
         let (mode, content) = value.split_once(':').context(InvalidConfigSnafu {
             value: value.to_string(),
             msg: "StaticUserProviderOption must be in format `<option>:<value>`",
@@ -85,11 +83,15 @@ impl StaticUserProvider {
                 value: mode.to_string(),
                 msg: "StaticUserProviderOption must be in format `file:<path>` or `cmd:<values>`",
             }
-                .fail(),
+            .fail(),
         };
     }
 }
 
+pub(crate) struct StaticUserProvider {
+    users: HashMap<String, Vec<u8>>,
+}
+
 #[async_trait]
 impl UserProvider for StaticUserProvider {
     fn name(&self) -> &str {
@@ -179,7 +181,7 @@ pub mod test {
     #[tokio::test]
     async fn test_authorize() {
         let user_info = DefaultUserInfo::with_name("root");
-        let provider = StaticUserProvider::new("cmd:root=123456,admin=654321").unwrap();
+        let provider = StaticUserProvider::try_from("cmd:root=123456,admin=654321").unwrap();
         provider
             .authorize("catalog", "schema", &user_info)
             .await
@@ -188,7 +190,7 @@ pub mod test {
 
     #[tokio::test]
     async fn test_inline_provider() {
-        let provider = StaticUserProvider::new("cmd:root=123456,admin=654321").unwrap();
+        let provider = StaticUserProvider::try_from("cmd:root=123456,admin=654321").unwrap();
         test_authenticate(&provider, "root", "123456").await;
         test_authenticate(&provider, "admin", "654321").await;
     }
@@ -212,7 +214,7 @@ admin=654321",
         }
 
         let param = format!("file:{file_path}");
-        let provider = StaticUserProvider::new(param.as_str()).unwrap();
+        let provider = StaticUserProvider::try_from(param.as_str()).unwrap();
         test_authenticate(&provider, "root", "123456").await;
         test_authenticate(&provider, "admin", "654321").await;
     }

src/catalog/Cargo.toml

@@ -7,9 +7,6 @@ license.workspace = true
 [features]
 testing = []
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 arc-swap = "1.0"

src/catalog/src/error.rs

@@ -164,8 +164,11 @@ pub enum Error {
         location: Location,
     },
 
-    #[snafu(display("Failed to find table partitions"))]
-    FindPartitions { source: partition::error::Error },
+    #[snafu(display("Failed to find table partitions: #{table}"))]
+    FindPartitions {
+        source: partition::error::Error,
+        table: String,
+    },
 
     #[snafu(display("Failed to find region routes"))]
     FindRegionRoutes { source: partition::error::Error },
@@ -251,12 +254,6 @@ pub enum Error {
         source: common_meta::error::Error,
         location: Location,
     },
-
-    #[snafu(display("Get null from table cache, key: {}", key))]
-    TableCacheNotGet { key: String, location: Location },
-
-    #[snafu(display("Failed to get table cache, err: {}", err_msg))]
-    GetTableCache { err_msg: String },
 }
 
 pub type Result<T> = std::result::Result<T, Error>;
@@ -317,7 +314,6 @@ impl ErrorExt for Error {
             Error::QueryAccessDenied { .. } => StatusCode::AccessDenied,
             Error::Datafusion { .. } => StatusCode::EngineExecuteQuery,
             Error::TableMetadataManager { source, .. } => source.status_code(),
-            Error::TableCacheNotGet { .. } | Error::GetTableCache { .. } => StatusCode::Internal,
         }
     }
 

src/catalog/src/information_schema.rs

@@ -19,9 +19,9 @@ mod partitions;
 mod predicate;
 mod region_peers;
 mod runtime_metrics;
-pub mod schemata;
+mod schemata;
 mod table_names;
-pub mod tables;
+mod tables;
 
 use std::collections::HashMap;
 use std::sync::{Arc, Weak};

src/catalog/src/information_schema/partitions.rs

@@ -12,7 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-use core::pin::pin;
 use std::sync::{Arc, Weak};
 
 use arrow_schema::SchemaRef as ArrowSchemaRef;
@@ -32,7 +31,7 @@ use datatypes::vectors::{
     ConstantVector, DateTimeVector, DateTimeVectorBuilder, Int64Vector, Int64VectorBuilder,
     MutableVector, StringVector, StringVectorBuilder, UInt64VectorBuilder,
 };
-use futures::{StreamExt, TryStreamExt};
+use futures::TryStreamExt;
 use partition::manager::PartitionInfo;
 use partition::partition::PartitionDef;
 use snafu::{OptionExt, ResultExt};
@@ -241,64 +240,40 @@ impl InformationSchemaPartitionsBuilder {
         let predicates = Predicates::from_scan_request(&request);
 
         for schema_name in catalog_manager.schema_names(&catalog_name).await? {
-            let table_info_stream = catalog_manager
-                .tables(&catalog_name, &schema_name)
-                .await
-                .try_filter_map(|t| async move {
-                    let table_info = t.table_info();
-                    if table_info.table_type == TableType::Temporary {
-                        Ok(None)
-                    } else {
-                        Ok(Some(table_info))
-                    }
-                });
+            let mut stream = catalog_manager.tables(&catalog_name, &schema_name).await;
 
-            const BATCH_SIZE: usize = 128;
+            while let Some(table) = stream.try_next().await? {
+                let table_info = table.table_info();
 
-            // Split table infos into chunks
-            let mut table_info_chunks = pin!(table_info_stream.ready_chunks(BATCH_SIZE));
+                if table_info.table_type == TableType::Temporary {
+                    continue;
+                }
 
-            while let Some(table_infos) = table_info_chunks.next().await {
-                let table_infos = table_infos.into_iter().collect::<Result<Vec<_>>>()?;
-                let table_ids: Vec<TableId> =
-                    table_infos.iter().map(|info| info.ident.table_id).collect();
-
-                let mut table_partitions = if let Some(partition_manager) = &partition_manager {
+                let table_id = table_info.ident.table_id;
+                let partitions = if let Some(partition_manager) = &partition_manager {
                     partition_manager
-                        .batch_find_table_partitions(&table_ids)
+                        .find_table_partitions(table_id)
                         .await
-                        .context(FindPartitionsSnafu)?
+                        .context(FindPartitionsSnafu {
+                            table: &table_info.name,
+                        })?
                 } else {
                     // Current node must be a standalone instance, contains only one partition by default.
                     // TODO(dennis): change it when we support multi-regions for standalone.
-                    table_ids
-                        .into_iter()
-                        .map(|table_id| {
-                            (
-                                table_id,
-                                vec![PartitionInfo {
-                                    id: RegionId::new(table_id, 0),
-                                    partition: PartitionDef::new(vec![], vec![]),
-                                }],
-                            )
-                        })
-                        .collect()
+                    vec![PartitionInfo {
+                        id: RegionId::new(table_id, 0),
+                        partition: PartitionDef::new(vec![], vec![]),
+                    }]
                 };
 
-                for table_info in table_infos {
-                    let partitions = table_partitions
-                        .remove(&table_info.ident.table_id)
-                        .unwrap_or(vec![]);
-
-                    self.add_partitions(
-                        &predicates,
-                        &table_info,
-                        &catalog_name,
-                        &schema_name,
-                        &table_info.name,
-                        &partitions,
-                    );
-                }
+                self.add_partitions(
+                    &predicates,
+                    &table_info,
+                    &catalog_name,
+                    &schema_name,
+                    &table_info.name,
+                    &partitions,
+                );
             }
         }
 

src/catalog/src/information_schema/region_peers.rs

@@ -199,7 +199,7 @@ impl InformationSchemaRegionPeersBuilder {
 
                 let table_routes = if let Some(partition_manager) = &partition_manager {
                     partition_manager
-                        .batch_find_region_routes(&table_ids)
+                        .find_region_routes_batch(&table_ids)
                         .await
                         .context(FindRegionRoutesSnafu)?
                 } else {

src/catalog/src/information_schema/schemata.rs

@@ -37,8 +37,8 @@ use crate::error::{
 use crate::information_schema::{InformationTable, Predicates};
 use crate::CatalogManager;
 
-pub const CATALOG_NAME: &str = "catalog_name";
-pub const SCHEMA_NAME: &str = "schema_name";
+const CATALOG_NAME: &str = "catalog_name";
+const SCHEMA_NAME: &str = "schema_name";
 const DEFAULT_CHARACTER_SET_NAME: &str = "default_character_set_name";
 const DEFAULT_COLLATION_NAME: &str = "default_collation_name";
 const INIT_CAPACITY: usize = 42;

src/catalog/src/information_schema/tables.rs

@@ -39,10 +39,10 @@ use crate::error::{
 use crate::information_schema::{InformationTable, Predicates};
 use crate::CatalogManager;
 
-pub const TABLE_CATALOG: &str = "table_catalog";
-pub const TABLE_SCHEMA: &str = "table_schema";
-pub const TABLE_NAME: &str = "table_name";
-pub const TABLE_TYPE: &str = "table_type";
+const TABLE_CATALOG: &str = "table_catalog";
+const TABLE_SCHEMA: &str = "table_schema";
+const TABLE_NAME: &str = "table_name";
+const TABLE_TYPE: &str = "table_type";
 const TABLE_ID: &str = "table_id";
 const ENGINE: &str = "engine";
 const INIT_CAPACITY: usize = 42;

src/catalog/src/kvbackend/client.rs

@@ -82,10 +82,12 @@ impl CachedMetaKvBackendBuilder {
         let cache_ttl = self.cache_ttl.unwrap_or(DEFAULT_CACHE_TTL);
         let cache_tti = self.cache_tti.unwrap_or(DEFAULT_CACHE_TTI);
 
-        let cache = CacheBuilder::new(cache_max_capacity)
-            .time_to_live(cache_ttl)
-            .time_to_idle(cache_tti)
-            .build();
+        let cache = Arc::new(
+            CacheBuilder::new(cache_max_capacity)
+                .time_to_live(cache_ttl)
+                .time_to_idle(cache_tti)
+                .build(),
+        );
 
         let kv_backend = Arc::new(MetaKvBackend {
             client: self.meta_client,
@@ -102,7 +104,7 @@ impl CachedMetaKvBackendBuilder {
     }
 }
 
-pub type CacheBackend = Cache<Vec<u8>, KeyValue>;
+pub type CacheBackendRef = Arc<Cache<Vec<u8>, KeyValue>>;
 
 /// A wrapper of `MetaKvBackend` with cache support.
 ///
@@ -115,7 +117,7 @@ pub type CacheBackend = Cache<Vec<u8>, KeyValue>;
 /// TTL and TTI for cache.
 pub struct CachedMetaKvBackend {
     kv_backend: KvBackendRef,
-    cache: CacheBackend,
+    cache: CacheBackendRef,
     name: String,
     version: AtomicUsize,
 }
@@ -315,10 +317,12 @@ impl CachedMetaKvBackend {
     // only for test
     #[cfg(test)]
     fn wrap(kv_backend: KvBackendRef) -> Self {
-        let cache = CacheBuilder::new(DEFAULT_CACHE_MAX_CAPACITY)
-            .time_to_live(DEFAULT_CACHE_TTL)
-            .time_to_idle(DEFAULT_CACHE_TTI)
-            .build();
+        let cache = Arc::new(
+            CacheBuilder::new(DEFAULT_CACHE_MAX_CAPACITY)
+                .time_to_live(DEFAULT_CACHE_TTL)
+                .time_to_idle(DEFAULT_CACHE_TTI)
+                .build(),
+        );
 
         let name = format!("CachedKvBackend({})", kv_backend.name());
         Self {
@@ -329,7 +333,7 @@ impl CachedMetaKvBackend {
         }
     }
 
-    pub fn cache(&self) -> &CacheBackend {
+    pub fn cache(&self) -> &CacheBackendRef {
         &self.cache
     }
 

src/catalog/src/kvbackend/manager.rs

@@ -15,37 +15,32 @@
 use std::any::Any;
 use std::collections::BTreeSet;
 use std::sync::{Arc, Weak};
-use std::time::Duration;
 
 use async_stream::try_stream;
-use common_catalog::consts::{
-    DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME, INFORMATION_SCHEMA_NAME, NUMBERS_TABLE_ID,
-};
-use common_catalog::format_full_table_name;
+use common_catalog::consts::{DEFAULT_SCHEMA_NAME, INFORMATION_SCHEMA_NAME, NUMBERS_TABLE_ID};
 use common_error::ext::BoxedError;
 use common_meta::cache_invalidator::{CacheInvalidator, CacheInvalidatorRef, Context};
 use common_meta::error::Result as MetaResult;
-use common_meta::instruction::CacheIdent;
 use common_meta::key::catalog_name::CatalogNameKey;
 use common_meta::key::schema_name::SchemaNameKey;
 use common_meta::key::table_info::TableInfoValue;
 use common_meta::key::table_name::TableNameKey;
 use common_meta::key::{TableMetadataManager, TableMetadataManagerRef};
 use common_meta::kv_backend::KvBackendRef;
+use common_meta::table_name::TableName;
 use futures_util::stream::BoxStream;
 use futures_util::{StreamExt, TryStreamExt};
-use moka::future::{Cache as AsyncCache, CacheBuilder};
 use moka::sync::Cache;
 use partition::manager::{PartitionRuleManager, PartitionRuleManagerRef};
 use snafu::prelude::*;
 use table::dist_table::DistTable;
+use table::metadata::TableId;
 use table::table::numbers::{NumbersTable, NUMBERS_TABLE_NAME};
 use table::TableRef;
 
-use crate::error::Error::{GetTableCache, TableCacheNotGet};
 use crate::error::{
     self as catalog_err, ListCatalogsSnafu, ListSchemasSnafu, ListTablesSnafu,
-    Result as CatalogResult, TableCacheNotGetSnafu, TableMetadataManagerSnafu,
+    Result as CatalogResult, TableMetadataManagerSnafu,
 };
 use crate::information_schema::InformationSchemaProvider;
 use crate::CatalogManager;
@@ -65,7 +60,6 @@ pub struct KvBackendCatalogManager {
     table_metadata_manager: TableMetadataManagerRef,
     /// A sub-CatalogManager that handles system tables
     system_catalog: SystemCatalog,
-    table_cache: AsyncCache<String, TableRef>,
 }
 
 fn make_table(table_info_value: TableInfoValue) -> CatalogResult<TableRef> {
@@ -78,25 +72,20 @@ fn make_table(table_info_value: TableInfoValue) -> CatalogResult<TableRef> {
 
 #[async_trait::async_trait]
 impl CacheInvalidator for KvBackendCatalogManager {
-    async fn invalidate(&self, ctx: &Context, caches: Vec<CacheIdent>) -> MetaResult<()> {
-        for cache in &caches {
-            if let CacheIdent::TableName(table_name) = cache {
-                let table_cache_key = format_full_table_name(
-                    &table_name.catalog_name,
-                    &table_name.schema_name,
-                    &table_name.table_name,
-                );
-                self.table_cache.invalidate(&table_cache_key).await;
-            }
-        }
-        self.cache_invalidator.invalidate(ctx, caches).await
+    async fn invalidate_table_id(&self, ctx: &Context, table_id: TableId) -> MetaResult<()> {
+        self.cache_invalidator
+            .invalidate_table_id(ctx, table_id)
+            .await
+    }
+
+    async fn invalidate_table_name(&self, ctx: &Context, table_name: TableName) -> MetaResult<()> {
+        self.cache_invalidator
+            .invalidate_table_name(ctx, table_name)
+            .await
     }
 }
 
-const CATALOG_CACHE_MAX_CAPACITY: u64 = 128;
-const TABLE_CACHE_MAX_CAPACITY: u64 = 65536;
-const TABLE_CACHE_TTL: Duration = Duration::from_secs(10 * 60);
-const TABLE_CACHE_TTI: Duration = Duration::from_secs(5 * 60);
+const DEFAULT_CACHED_CATALOG: u64 = 128;
 
 impl KvBackendCatalogManager {
     pub fn new(backend: KvBackendRef, cache_invalidator: CacheInvalidatorRef) -> Arc<Self> {
@@ -106,16 +95,13 @@ impl KvBackendCatalogManager {
             cache_invalidator,
             system_catalog: SystemCatalog {
                 catalog_manager: me.clone(),
-                catalog_cache: Cache::new(CATALOG_CACHE_MAX_CAPACITY),
+                catalog_cache: Cache::new(DEFAULT_CACHED_CATALOG),
                 information_schema_provider: Arc::new(InformationSchemaProvider::new(
-                    DEFAULT_CATALOG_NAME.to_string(),
+                    // The catalog name is not used in system_catalog, so let it empty
+                    String::default(),
                     me.clone(),
                 )),
             },
-            table_cache: CacheBuilder::new(TABLE_CACHE_MAX_CAPACITY)
-                .time_to_live(TABLE_CACHE_TTL)
-                .time_to_idle(TABLE_CACHE_TTI)
-                .build(),
         })
     }
 
@@ -230,52 +216,29 @@ impl CatalogManager for KvBackendCatalogManager {
             return Ok(Some(table));
         }
 
-        let init = async {
-            let table_name_key = TableNameKey::new(catalog, schema, table_name);
-            let Some(table_name_value) = self
-                .table_metadata_manager
-                .table_name_manager()
-                .get(table_name_key)
-                .await
-                .context(TableMetadataManagerSnafu)?
-            else {
-                return TableCacheNotGetSnafu {
-                    key: table_name_key.to_string(),
-                }
-                .fail();
-            };
-            let table_id = table_name_value.table_id();
-
-            let Some(table_info_value) = self
-                .table_metadata_manager
-                .table_info_manager()
-                .get(table_id)
-                .await
-                .context(TableMetadataManagerSnafu)?
-                .map(|v| v.into_inner())
-            else {
-                return TableCacheNotGetSnafu {
-                    key: table_name_key.to_string(),
-                }
-                .fail();
-            };
-            make_table(table_info_value)
-        };
-
-        match self
-            .table_cache
-            .try_get_with_by_ref(&format_full_table_name(catalog, schema, table_name), init)
+        let key = TableNameKey::new(catalog, schema, table_name);
+        let Some(table_name_value) = self
+            .table_metadata_manager
+            .table_name_manager()
+            .get(key)
             .await
-        {
-            Ok(table) => Ok(Some(table)),
-            Err(err) => match err.as_ref() {
-                TableCacheNotGet { .. } => Ok(None),
-                _ => Err(err),
-            },
-        }
-        .map_err(|err| GetTableCache {
-            err_msg: err.to_string(),
-        })
+            .context(TableMetadataManagerSnafu)?
+        else {
+            return Ok(None);
+        };
+        let table_id = table_name_value.table_id();
+
+        let Some(table_info_value) = self
+            .table_metadata_manager
+            .table_info_manager()
+            .get(table_id)
+            .await
+            .context(TableMetadataManagerSnafu)?
+            .map(|v| v.into_inner())
+        else {
+            return Ok(None);
+        };
+        make_table(table_info_value).map(Some)
     }
 
     async fn tables<'a>(

src/client/Cargo.toml

@@ -7,9 +7,6 @@ license.workspace = true
 [features]
 testing = []
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 arc-swap = "1.6"
@@ -37,8 +34,6 @@ parking_lot = "0.12"
 prometheus.workspace = true
 prost.workspace = true
 rand.workspace = true
-serde.workspace = true
-serde_json.workspace = true
 session.workspace = true
 snafu.workspace = true
 tokio.workspace = true

src/client/src/database.rs

@@ -307,7 +307,7 @@ impl Database {
                         reason: "Expect 'AffectedRows' Flight messages to be the one and the only!"
                     }
                 );
-                Ok(Output::new_with_affected_rows(rows))
+                Ok(Output::AffectedRows(rows))
             }
             FlightMessage::Recordbatch(_) | FlightMessage::Metrics(_) => {
                 IllegalFlightMessagesSnafu {
@@ -340,7 +340,7 @@ impl Database {
                     output_ordering: None,
                     metrics: Default::default(),
                 };
-                Ok(Output::new_with_stream(Box::pin(record_batch_stream)))
+                Ok(Output::Stream(Box::pin(record_batch_stream)))
             }
         }
     }

src/client/src/error.rs

@@ -134,17 +134,10 @@ impl From<Status> for Error {
 
 impl Error {
     pub fn should_retry(&self) -> bool {
-        // TODO(weny): figure out each case of these codes.
-        matches!(
+        !matches!(
             self,
             Self::RegionServer {
-                code: Code::Cancelled,
-                ..
-            } | Self::RegionServer {
-                code: Code::DeadlineExceeded,
-                ..
-            } | Self::RegionServer {
-                code: Code::Unavailable,
+                code: Code::InvalidArgument,
                 ..
             }
         )

src/client/src/lib.rs

@@ -26,7 +26,7 @@ use api::v1::greptime_response::Response;
 use api::v1::{AffectedRows, GreptimeResponse};
 pub use common_catalog::consts::{DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
 use common_error::status_code::StatusCode;
-pub use common_query::{Output, OutputData, OutputMeta};
+pub use common_query::Output;
 pub use common_recordbatch::{RecordBatches, SendableRecordBatchStream};
 use snafu::OptionExt;
 

src/client/src/region.rs

@@ -14,7 +14,7 @@
 
 use std::sync::Arc;
 
-use api::v1::region::{QueryRequest, RegionRequest};
+use api::v1::region::{QueryRequest, RegionRequest, RegionResponse};
 use api::v1::ResponseHeader;
 use arc_swap::ArcSwapOption;
 use arrow_flight::Ticket;
@@ -23,7 +23,7 @@ use async_trait::async_trait;
 use common_error::ext::{BoxedError, ErrorExt};
 use common_error::status_code::StatusCode;
 use common_grpc::flight::{FlightDecoder, FlightMessage};
-use common_meta::datanode_manager::{Datanode, HandleResponse};
+use common_meta::datanode_manager::{AffectedRows, Datanode};
 use common_meta::error::{self as meta_error, Result as MetaResult};
 use common_recordbatch::error::ExternalSnafu;
 use common_recordbatch::{RecordBatchStreamWrapper, SendableRecordBatchStream};
@@ -46,7 +46,7 @@ pub struct RegionRequester {
 
 #[async_trait]
 impl Datanode for RegionRequester {
-    async fn handle(&self, request: RegionRequest) -> MetaResult<HandleResponse> {
+    async fn handle(&self, request: RegionRequest) -> MetaResult<AffectedRows> {
         self.handle_inner(request).await.map_err(|err| {
             if err.should_retry() {
                 meta_error::Error::RetryLater {
@@ -123,8 +123,8 @@ impl RegionRequester {
             .fail();
         };
 
-        let metrics = Arc::new(ArcSwapOption::from(None));
-        let metrics_ref = metrics.clone();
+        let metrics_str = Arc::new(ArcSwapOption::from(None));
+        let ref_str = metrics_str.clone();
 
         let tracing_context = TracingContext::from_current_span();
 
@@ -140,8 +140,7 @@ impl RegionRequester {
                 match flight_message {
                     FlightMessage::Recordbatch(record_batch) => yield Ok(record_batch),
                     FlightMessage::Metrics(s) => {
-                        let m = serde_json::from_str(&s).ok().map(Arc::new);
-                        metrics_ref.swap(m);
+                        ref_str.swap(Some(Arc::new(s)));
                         break;
                     }
                     _ => {
@@ -160,12 +159,12 @@ impl RegionRequester {
             schema,
             stream,
             output_ordering: None,
-            metrics,
+            metrics: metrics_str,
         };
         Ok(Box::pin(record_batch_stream))
     }
 
-    async fn handle_inner(&self, request: RegionRequest) -> Result<HandleResponse> {
+    async fn handle_inner(&self, request: RegionRequest) -> Result<AffectedRows> {
         let request_type = request
             .body
             .as_ref()
@@ -178,7 +177,10 @@ impl RegionRequester {
 
         let mut client = self.client.raw_region_client()?;
 
-        let response = client
+        let RegionResponse {
+            header,
+            affected_rows,
+        } = client
             .handle(request)
             .await
             .map_err(|e| {
@@ -192,20 +194,19 @@ impl RegionRequester {
             })?
             .into_inner();
 
-        check_response_header(&response.header)?;
+        check_response_header(header)?;
 
-        Ok(HandleResponse::from_region_response(response))
+        Ok(affected_rows)
     }
 
-    pub async fn handle(&self, request: RegionRequest) -> Result<HandleResponse> {
+    pub async fn handle(&self, request: RegionRequest) -> Result<AffectedRows> {
         self.handle_inner(request).await
     }
 }
 
-pub fn check_response_header(header: &Option<ResponseHeader>) -> Result<()> {
+pub fn check_response_header(header: Option<ResponseHeader>) -> Result<()> {
     let status = header
-        .as_ref()
-        .and_then(|header| header.status.as_ref())
+        .and_then(|header| header.status)
         .context(IllegalDatabaseResponseSnafu {
             err_msg: "either response header or status is missing",
         })?;
@@ -219,7 +220,7 @@ pub fn check_response_header(header: &Option<ResponseHeader>) -> Result<()> {
             })?;
         ServerSnafu {
             code,
-            msg: status.err_msg.clone(),
+            msg: status.err_msg,
         }
         .fail()
     }
@@ -234,19 +235,19 @@ mod test {
 
     #[test]
     fn test_check_response_header() {
-        let result = check_response_header(&None);
+        let result = check_response_header(None);
         assert!(matches!(
             result.unwrap_err(),
             IllegalDatabaseResponse { .. }
         ));
 
-        let result = check_response_header(&Some(ResponseHeader { status: None }));
+        let result = check_response_header(Some(ResponseHeader { status: None }));
         assert!(matches!(
             result.unwrap_err(),
             IllegalDatabaseResponse { .. }
         ));
 
-        let result = check_response_header(&Some(ResponseHeader {
+        let result = check_response_header(Some(ResponseHeader {
             status: Some(PbStatus {
                 status_code: StatusCode::Success as u32,
                 err_msg: String::default(),
@@ -254,7 +255,7 @@ mod test {
         }));
         assert!(result.is_ok());
 
-        let result = check_response_header(&Some(ResponseHeader {
+        let result = check_response_header(Some(ResponseHeader {
             status: Some(PbStatus {
                 status_code: u32::MAX,
                 err_msg: String::default(),
@@ -265,7 +266,7 @@ mod test {
             IllegalDatabaseResponse { .. }
         ));
 
-        let result = check_response_header(&Some(ResponseHeader {
+        let result = check_response_header(Some(ResponseHeader {
             status: Some(PbStatus {
                 status_code: StatusCode::Internal as u32,
                 err_msg: "blabla".to_string(),

src/cmd/Cargo.toml

@@ -12,9 +12,6 @@ path = "src/bin/greptime.rs"
 [features]
 tokio-console = ["common-telemetry/tokio-console"]
 
-[lints]
-workspace = true
-
 [dependencies]
 anymap = "1.0.0-beta.2"
 async-trait.workspace = true

src/cmd/src/cli/bench.rs

@@ -62,9 +62,7 @@ pub struct BenchTableMetadataCommand {
 
 impl BenchTableMetadataCommand {
     pub async fn build(&self) -> Result<Instance> {
-        let etcd_store = EtcdStore::with_endpoints([&self.etcd_addr], 128)
-            .await
-            .unwrap();
+        let etcd_store = EtcdStore::with_endpoints([&self.etcd_addr]).await.unwrap();
 
         let table_metadata_manager = Arc::new(TableMetadataManager::new(etcd_store));
 

src/cmd/src/cli/export.rs

@@ -19,7 +19,8 @@ use async_trait::async_trait;
 use clap::{Parser, ValueEnum};
 use client::api::v1::auth_header::AuthScheme;
 use client::api::v1::Basic;
-use client::{Client, Database, OutputData, DEFAULT_SCHEMA_NAME};
+use client::{Client, Database, DEFAULT_SCHEMA_NAME};
+use common_query::Output;
 use common_recordbatch::util::collect;
 use common_telemetry::{debug, error, info, warn};
 use datatypes::scalars::ScalarVector;
@@ -141,7 +142,7 @@ impl Export {
                     .with_context(|_| RequestDatabaseSnafu {
                         sql: "show databases".to_string(),
                     })?;
-            let OutputData::Stream(stream) = result.data else {
+            let Output::Stream(stream) = result else {
                 NotDataFromOutputSnafu.fail()?
             };
             let record_batch = collect(stream)
@@ -182,7 +183,7 @@ impl Export {
             .sql(&sql)
             .await
             .with_context(|_| RequestDatabaseSnafu { sql })?;
-        let OutputData::Stream(stream) = result.data else {
+        let Output::Stream(stream) = result else {
             NotDataFromOutputSnafu.fail()?
         };
         let Some(record_batch) = collect(stream)
@@ -234,7 +235,7 @@ impl Export {
             .sql(&sql)
             .await
             .with_context(|_| RequestDatabaseSnafu { sql })?;
-        let OutputData::Stream(stream) = result.data else {
+        let Output::Stream(stream) = result else {
             NotDataFromOutputSnafu.fail()?
         };
         let record_batch = collect(stream)

src/cmd/src/cli/repl.rs

@@ -19,7 +19,7 @@ use std::time::Instant;
 use catalog::kvbackend::{
     CachedMetaKvBackend, CachedMetaKvBackendBuilder, KvBackendCatalogManager,
 };
-use client::{Client, Database, OutputData, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
+use client::{Client, Database, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
 use common_base::Plugins;
 use common_error::ext::ErrorExt;
 use common_query::Output;
@@ -184,15 +184,15 @@ impl Repl {
         }
         .context(RequestDatabaseSnafu { sql: &sql })?;
 
-        let either = match output.data {
-            OutputData::Stream(s) => {
+        let either = match output {
+            Output::Stream(s) => {
                 let x = RecordBatches::try_collect(s)
                     .await
                     .context(CollectRecordBatchesSnafu)?;
                 Either::Left(x)
             }
-            OutputData::RecordBatches(x) => Either::Left(x),
-            OutputData::AffectedRows(rows) => Either::Right(rows),
+            Output::RecordBatches(x) => Either::Left(x),
+            Output::AffectedRows(rows) => Either::Right(rows),
         };
 
         let end = Instant::now();
@@ -260,7 +260,6 @@ async fn create_query_engine(meta_addr: &str) -> Result<DatafusionQueryEngine> {
         catalog_list,
         None,
         None,
-        None,
         false,
         plugins.clone(),
     ));

src/cmd/src/cli/upgrade.rs

@@ -70,7 +70,7 @@ impl UpgradeCommand {
                 etcd_addr: &self.etcd_addr,
             })?;
         let tool = MigrateTableMetadata {
-            etcd_store: EtcdStore::with_etcd_client(client, 128),
+            etcd_store: EtcdStore::with_etcd_client(client),
             dryrun: self.dryrun,
             skip_catalog_keys: self.skip_catalog_keys,
             skip_table_global_keys: self.skip_table_global_keys,

src/cmd/src/datanode.rs

@@ -43,10 +43,6 @@ impl Instance {
     pub fn datanode_mut(&mut self) -> &mut Datanode {
         &mut self.datanode
     }
-
-    pub fn datanode(&self) -> &Datanode {
-        &self.datanode
-    }
 }
 
 #[async_trait]
@@ -239,7 +235,6 @@ impl StartCommand {
             .with_default_grpc_server(&datanode.region_server())
             .enable_http_service()
             .build()
-            .await
             .context(StartDatanodeSnafu)?;
         datanode.setup_services(services);
 

src/cmd/src/frontend.rs

@@ -43,17 +43,13 @@ pub struct Instance {
 }
 
 impl Instance {
-    pub fn new(frontend: FeInstance) -> Self {
+    fn new(frontend: FeInstance) -> Self {
         Self { frontend }
     }
 
     pub fn mut_inner(&mut self) -> &mut FeInstance {
         &mut self.frontend
     }
-
-    pub fn inner(&self) -> &FeInstance {
-        &self.frontend
-    }
 }
 
 #[async_trait]
@@ -275,7 +271,6 @@ impl StartCommand {
 
         let servers = Services::new(opts.clone(), Arc::new(instance.clone()), plugins)
             .build()
-            .await
             .context(StartFrontendSnafu)?;
         instance
             .build_servers(opts, servers)

src/cmd/src/lib.rs

@@ -32,11 +32,11 @@ lazy_static::lazy_static! {
 }
 
 #[async_trait]
-pub trait App: Send {
+pub trait App {
     fn name(&self) -> &str;
 
     /// A hook for implementor to make something happened before actual startup. Defaults to no-op.
-    async fn pre_start(&mut self) -> error::Result<()> {
+    fn pre_start(&mut self) -> error::Result<()> {
         Ok(())
     }
 
@@ -46,21 +46,24 @@ pub trait App: Send {
 }
 
 pub async fn start_app(mut app: Box<dyn App>) -> error::Result<()> {
-    info!("Starting app: {}", app.name());
+    let name = app.name().to_string();
 
-    app.pre_start().await?;
+    app.pre_start()?;
 
-    app.start().await?;
-
-    if let Err(e) = tokio::signal::ctrl_c().await {
-        error!("Failed to listen for ctrl-c signal: {}", e);
-        // It's unusual to fail to listen for ctrl-c signal, maybe there's something unexpected in
-        // the underlying system. So we stop the app instead of running nonetheless to let people
-        // investigate the issue.
+    tokio::select! {
+        result = app.start() => {
+            if let Err(err) = result {
+                error!(err; "Failed to start app {name}!");
+            }
+        }
+        _ = tokio::signal::ctrl_c() => {
+            if let Err(err) = app.stop().await {
+                error!(err; "Failed to stop app {name}!");
+            }
+            info!("Goodbye!");
+        }
     }
 
-    app.stop().await?;
-    info!("Goodbye!");
     Ok(())
 }
 

src/cmd/src/metasrv.rs

@@ -117,12 +117,10 @@ struct StartCommand {
     /// The working home directory of this metasrv instance.
     #[clap(long)]
     data_home: Option<String>,
+
     /// If it's not empty, the metasrv will store all data with this key prefix.
     #[clap(long, default_value = "")]
     store_key_prefix: String,
-    /// The max operations per txn
-    #[clap(long)]
-    max_txn_ops: Option<usize>,
 }
 
 impl StartCommand {
@@ -183,10 +181,6 @@ impl StartCommand {
             opts.store_key_prefix = self.store_key_prefix.clone()
         }
 
-        if let Some(max_txn_ops) = self.max_txn_ops {
-            opts.max_txn_ops = max_txn_ops;
-        }
-
         // Disable dashboard in metasrv.
         opts.http.disable_dashboard = true;
 

src/cmd/src/standalone.rs

@@ -21,8 +21,8 @@ use common_catalog::consts::MIN_USER_TABLE_ID;
 use common_config::{metadata_store_dir, KvBackendConfig};
 use common_meta::cache_invalidator::DummyCacheInvalidator;
 use common_meta::datanode_manager::DatanodeManagerRef;
-use common_meta::ddl::table_meta::{TableMetadataAllocator, TableMetadataAllocatorRef};
-use common_meta::ddl::ProcedureExecutorRef;
+use common_meta::ddl::table_meta::TableMetadataAllocator;
+use common_meta::ddl::DdlTaskExecutorRef;
 use common_meta::ddl_manager::DdlManager;
 use common_meta::key::{TableMetadataManager, TableMetadataManagerRef};
 use common_meta::kv_backend::KvBackendRef;
@@ -419,11 +419,11 @@ impl StartCommand {
         let table_metadata_manager =
             Self::create_table_metadata_manager(kv_backend.clone()).await?;
 
-        let table_meta_allocator = Arc::new(TableMetadataAllocator::new(
+        let table_meta_allocator = TableMetadataAllocator::new(
             table_id_sequence,
             wal_options_allocator.clone(),
-            table_metadata_manager.table_name_manager().clone(),
-        ));
+            table_metadata_manager.clone(),
+        );
 
         let ddl_task_executor = Self::create_ddl_task_executor(
             table_metadata_manager,
@@ -441,7 +441,6 @@ impl StartCommand {
 
         let servers = Services::new(fe_opts.clone(), Arc::new(frontend.clone()), fe_plugins)
             .build()
-            .await
             .context(StartFrontendSnafu)?;
         frontend
             .build_servers(fe_opts, servers)
@@ -459,9 +458,9 @@ impl StartCommand {
         table_metadata_manager: TableMetadataManagerRef,
         procedure_manager: ProcedureManagerRef,
         datanode_manager: DatanodeManagerRef,
-        table_meta_allocator: TableMetadataAllocatorRef,
-    ) -> Result<ProcedureExecutorRef> {
-        let procedure_executor: ProcedureExecutorRef = Arc::new(
+        table_meta_allocator: TableMetadataAllocator,
+    ) -> Result<DdlTaskExecutorRef> {
+        let ddl_task_executor: DdlTaskExecutorRef = Arc::new(
             DdlManager::try_new(
                 procedure_manager,
                 datanode_manager,
@@ -473,7 +472,7 @@ impl StartCommand {
             .context(InitDdlManagerSnafu)?,
         );
 
-        Ok(procedure_executor)
+        Ok(ddl_task_executor)
     }
 
     pub async fn create_table_metadata_manager(

src/common/base/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 anymap = "1.0.0-beta.2"
 bitvec = "1.0"

src/common/base/src/lib.rs

@@ -21,8 +21,6 @@ pub mod readable_size;
 use core::any::Any;
 use std::sync::{Arc, Mutex, MutexGuard};
 
-pub type AffectedRows = usize;
-
 pub use bit_vec::BitVec;
 
 /// [`Plugins`] is a wrapper of Arc contents.

src/common/catalog/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 common-error.workspace = true
 common-macro.workspace = true

src/common/config/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 common-base.workspace = true
 humantime-serde.workspace = true

src/common/datasource/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 arrow.workspace = true
 arrow-schema.workspace = true

src/common/datasource/src/object_store/s3.rs

@@ -28,15 +28,12 @@ const REGION: &str = "region";
 const ENABLE_VIRTUAL_HOST_STYLE: &str = "enable_virtual_host_style";
 
 pub fn is_supported_in_s3(key: &str) -> bool {
-    [
-        ENDPOINT,
-        ACCESS_KEY_ID,
-        SECRET_ACCESS_KEY,
-        SESSION_TOKEN,
-        REGION,
-        ENABLE_VIRTUAL_HOST_STYLE,
-    ]
-    .contains(&key)
+    key == ENDPOINT
+        || key == ACCESS_KEY_ID
+        || key == SECRET_ACCESS_KEY
+        || key == SESSION_TOKEN
+        || key == REGION
+        || key == ENABLE_VIRTUAL_HOST_STYLE
 }
 
 pub fn build_s3_backend(

src/common/decimal/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 arrow.workspace = true
 bigdecimal.workspace = true

src/common/error/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 snafu.workspace = true
 strum.workspace = true

src/common/error/src/lib.rs

@@ -19,9 +19,7 @@ pub mod format;
 pub mod mock;
 pub mod status_code;
 
-pub use snafu;
-
-// HACK - these headers are here for shared in gRPC services. For common HTTP headers,
-// please define in `src/servers/src/http/header.rs`.
 pub const GREPTIME_DB_HEADER_ERROR_CODE: &str = "x-greptime-err-code";
 pub const GREPTIME_DB_HEADER_ERROR_MSG: &str = "x-greptime-err-msg";
+
+pub use snafu;

src/common/function/Cargo.toml

@@ -4,19 +4,13 @@ edition.workspace = true
 version.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 arc-swap = "1.0"
 async-trait.workspace = true
 chrono-tz = "0.6"
-common-base.workspace = true
-common-catalog.workspace = true
 common-error.workspace = true
 common-macro.workspace = true
-common-meta.workspace = true
 common-query.workspace = true
 common-runtime.workspace = true
 common-telemetry.workspace = true
@@ -29,12 +23,9 @@ num = "0.4"
 num-traits = "0.2"
 once_cell.workspace = true
 paste = "1.0"
-serde.workspace = true
-serde_json.workspace = true
 session.workspace = true
 snafu.workspace = true
 statrs = "0.16"
-store-api.workspace = true
 table.workspace = true
 
 [dev-dependencies]

src/common/function/src/function.rs

@@ -30,17 +30,6 @@ pub struct FunctionContext {
     pub state: Arc<FunctionState>,
 }
 
-impl FunctionContext {
-    /// Create a mock [`FunctionContext`] for test.
-    #[cfg(any(test, feature = "testing"))]
-    pub fn mock() -> Self {
-        Self {
-            query_ctx: QueryContextBuilder::default().build(),
-            state: Arc::new(FunctionState::mock()),
-        }
-    }
-}
-
 impl Default for FunctionContext {
     fn default() -> Self {
         Self {

src/common/function/src/function_registry.rs

@@ -21,7 +21,6 @@ use once_cell::sync::Lazy;
 use crate::function::FunctionRef;
 use crate::scalars::aggregate::{AggregateFunctionMetaRef, AggregateFunctions};
 use crate::scalars::date::DateFunction;
-use crate::scalars::expression::ExpressionFunction;
 use crate::scalars::math::MathFunction;
 use crate::scalars::numpy::NumpyFunction;
 use crate::scalars::timestamp::TimestampFunction;
@@ -81,7 +80,6 @@ pub static FUNCTION_REGISTRY: Lazy<Arc<FunctionRegistry>> = Lazy::new(|| {
     NumpyFunction::register(&function_registry);
     TimestampFunction::register(&function_registry);
     DateFunction::register(&function_registry);
-    ExpressionFunction::register(&function_registry);
 
     // Aggregate functions
     AggregateFunctions::register(&function_registry);

src/common/function/src/handlers.rs

@@ -13,58 +13,42 @@
 // limitations under the License.
 
 use std::sync::Arc;
+use std::time::Duration;
 
+use api::v1::meta::ProcedureStateResponse;
 use async_trait::async_trait;
-use common_base::AffectedRows;
-use common_meta::rpc::procedure::{MigrateRegionRequest, ProcedureStateResponse};
 use common_query::error::Result;
-use common_query::Output;
 use session::context::QueryContextRef;
-use store_api::storage::RegionId;
-use table::requests::{CompactTableRequest, DeleteRequest, FlushTableRequest, InsertRequest};
+use table::requests::{DeleteRequest, InsertRequest};
+
+pub type AffectedRows = usize;
 
 /// A trait for handling table mutations in `QueryEngine`.
 #[async_trait]
 pub trait TableMutationHandler: Send + Sync {
     /// Inserts rows into the table.
-    async fn insert(&self, request: InsertRequest, ctx: QueryContextRef) -> Result<Output>;
+    async fn insert(&self, request: InsertRequest, ctx: QueryContextRef) -> Result<AffectedRows>;
 
     /// Delete rows from the table.
     async fn delete(&self, request: DeleteRequest, ctx: QueryContextRef) -> Result<AffectedRows>;
 
-    /// Trigger a flush task for table.
-    async fn flush(&self, request: FlushTableRequest, ctx: QueryContextRef)
-        -> Result<AffectedRows>;
-
-    /// Trigger a compaction task for table.
-    async fn compact(
+    /// Migrate a region from source peer to target peer, returns the procedure id if success.
+    async fn migrate_region(
         &self,
-        request: CompactTableRequest,
-        ctx: QueryContextRef,
-    ) -> Result<AffectedRows>;
-
-    /// Trigger a flush task for a table region.
-    async fn flush_region(&self, region_id: RegionId, ctx: QueryContextRef)
-        -> Result<AffectedRows>;
-
-    /// Trigger a compaction task for a table region.
-    async fn compact_region(
-        &self,
-        region_id: RegionId,
-        ctx: QueryContextRef,
-    ) -> Result<AffectedRows>;
+        region_id: u64,
+        from_peer: u64,
+        to_peer: u64,
+        replay_timeout: Duration,
+    ) -> Result<String>;
 }
 
-/// A trait for handling procedure service requests in `QueryEngine`.
+/// A trait for handling meta service requests in `QueryEngine`.
 #[async_trait]
-pub trait ProcedureServiceHandler: Send + Sync {
-    /// Migrate a region from source peer to target peer, returns the procedure id if success.
-    async fn migrate_region(&self, request: MigrateRegionRequest) -> Result<Option<String>>;
-
+pub trait MetaServiceHandler: Send + Sync {
     /// Query the procedure' state by its id
     async fn query_procedure_state(&self, pid: &str) -> Result<ProcedureStateResponse>;
 }
 
 pub type TableMutationHandlerRef = Arc<dyn TableMutationHandler>;
 
-pub type ProcedureServiceHandlerRef = Arc<dyn ProcedureServiceHandler>;
+pub type MetaServiceHandlerRef = Arc<dyn MetaServiceHandler>;

src/common/function/src/helper.rs

@@ -12,12 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-use common_query::error::{InvalidInputTypeSnafu, Result};
 use common_query::prelude::{Signature, TypeSignature, Volatility};
 use datatypes::prelude::ConcreteDataType;
-use datatypes::types::cast::cast;
-use datatypes::value::ValueRef;
-use snafu::ResultExt;
 
 /// Create a function signature with oneof signatures of interleaving two arguments.
 pub fn one_of_sigs2(args1: Vec<ConcreteDataType>, args2: Vec<ConcreteDataType>) -> Signature {
@@ -31,15 +27,3 @@ pub fn one_of_sigs2(args1: Vec<ConcreteDataType>, args2: Vec<ConcreteDataType>)
 
     Signature::one_of(sigs, Volatility::Immutable)
 }
-
-/// Cast a [`ValueRef`] to u64, returns `None` if fails
-pub fn cast_u64(value: &ValueRef) -> Result<Option<u64>> {
-    cast((*value).into(), &ConcreteDataType::uint64_datatype())
-        .context(InvalidInputTypeSnafu {
-            err_msg: format!(
-                "Failed to cast input into uint64, actual type: {:#?}",
-                value.data_type(),
-            ),
-        })
-        .map(|v| v.as_u64())
-}

src/common/function/src/lib.rs

@@ -12,7 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-mod macros;
 pub mod scalars;
 mod system;
 mod table;

src/common/function/src/macros.rs

@@ -1,27 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-/// Ensure current function is invokded under `greptime` catalog.
-#[macro_export]
-macro_rules! ensure_greptime {
-    ($func_ctx: expr) => {{
-        use common_catalog::consts::DEFAULT_CATALOG_NAME;
-        snafu::ensure!(
-            $func_ctx.query_ctx.current_catalog() == DEFAULT_CATALOG_NAME,
-            common_query::error::PermissionDeniedSnafu {
-                err_msg: format!("current catalog is not {DEFAULT_CATALOG_NAME}")
-            }
-        );
-    }};
-}

src/common/function/src/scalars/expression.rs

@@ -14,22 +14,8 @@
 
 mod binary;
 mod ctx;
-mod is_null;
 mod unary;
 
-use std::sync::Arc;
-
 pub use binary::scalar_binary_op;
 pub use ctx::EvalContext;
 pub use unary::scalar_unary_op;
-
-use crate::function_registry::FunctionRegistry;
-use crate::scalars::expression::is_null::IsNullFunction;
-
-pub(crate) struct ExpressionFunction;
-
-impl ExpressionFunction {
-    pub fn register(registry: &FunctionRegistry) {
-        registry.register(Arc::new(IsNullFunction));
-    }
-}

src/common/function/src/scalars/expression/is_null.rs

@@ -1,109 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt;
-use std::fmt::Display;
-use std::sync::Arc;
-
-use common_query::error;
-use common_query::error::{ArrowComputeSnafu, InvalidFuncArgsSnafu};
-use common_query::prelude::{Signature, Volatility};
-use datafusion::arrow::array::ArrayRef;
-use datafusion::arrow::compute::is_null;
-use datatypes::data_type::ConcreteDataType;
-use datatypes::prelude::VectorRef;
-use datatypes::vectors::Helper;
-use snafu::{ensure, ResultExt};
-
-use crate::function::{Function, FunctionContext};
-
-const NAME: &str = "isnull";
-
-/// The function to check whether an expression is NULL
-#[derive(Clone, Debug, Default)]
-pub struct IsNullFunction;
-
-impl Display for IsNullFunction {
-    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-        write!(f, "{}", NAME.to_ascii_uppercase())
-    }
-}
-
-impl Function for IsNullFunction {
-    fn name(&self) -> &str {
-        NAME
-    }
-
-    fn return_type(&self, _: &[ConcreteDataType]) -> common_query::error::Result<ConcreteDataType> {
-        Ok(ConcreteDataType::boolean_datatype())
-    }
-
-    fn signature(&self) -> Signature {
-        Signature::any(1, Volatility::Immutable)
-    }
-
-    fn eval(
-        &self,
-        _func_ctx: FunctionContext,
-        columns: &[VectorRef],
-    ) -> common_query::error::Result<VectorRef> {
-        ensure!(
-            columns.len() == 1,
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The length of the args is not correct, expect exactly one, have: {}",
-                    columns.len()
-                ),
-            }
-        );
-        let values = &columns[0];
-        let arrow_array = &values.to_arrow_array();
-        let result = is_null(arrow_array).context(ArrowComputeSnafu)?;
-
-        Helper::try_into_vector(Arc::new(result) as ArrayRef).context(error::FromArrowArraySnafu)
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use std::sync::Arc;
-
-    use common_query::prelude::TypeSignature;
-    use datatypes::scalars::ScalarVector;
-    use datatypes::vectors::{BooleanVector, Float32Vector};
-
-    use super::*;
-    #[test]
-    fn test_is_null_function() {
-        let is_null = IsNullFunction;
-        assert_eq!("isnull", is_null.name());
-        assert_eq!(
-            ConcreteDataType::boolean_datatype(),
-            is_null.return_type(&[]).unwrap()
-        );
-        assert_eq!(
-            is_null.signature(),
-            Signature {
-                type_signature: TypeSignature::Any(1),
-                volatility: Volatility::Immutable
-            }
-        );
-        let values = vec![None, Some(3.0), None];
-
-        let args: Vec<VectorRef> = vec![Arc::new(Float32Vector::from(values))];
-        let vector = is_null.eval(FunctionContext::default(), &args).unwrap();
-        let expect: VectorRef = Arc::new(BooleanVector::from_vec(vec![true, false, true]));
-        assert_eq!(expect, vector);
-    }
-}

src/common/function/src/scalars/math.rs

@@ -12,7 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-mod clamp;
 mod modulo;
 mod pow;
 mod rate;
@@ -20,7 +19,6 @@ mod rate;
 use std::fmt;
 use std::sync::Arc;
 
-pub use clamp::ClampFunction;
 use common_query::error::{GeneralDataFusionSnafu, Result};
 use common_query::prelude::Signature;
 use datafusion::error::DataFusionError;
@@ -42,8 +40,7 @@ impl MathFunction {
         registry.register(Arc::new(ModuloFunction));
         registry.register(Arc::new(PowFunction));
         registry.register(Arc::new(RateFunction));
-        registry.register(Arc::new(RangeFunction));
-        registry.register(Arc::new(ClampFunction));
+        registry.register(Arc::new(RangeFunction))
     }
 }
 

src/common/function/src/scalars/math/clamp.rs

@@ -1,403 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt::{self, Display};
-use std::sync::Arc;
-
-use common_query::error::{InvalidFuncArgsSnafu, Result};
-use common_query::prelude::Signature;
-use datafusion::arrow::array::{ArrayIter, PrimitiveArray};
-use datafusion::logical_expr::Volatility;
-use datatypes::data_type::{ConcreteDataType, DataType};
-use datatypes::prelude::VectorRef;
-use datatypes::types::LogicalPrimitiveType;
-use datatypes::value::TryAsPrimitive;
-use datatypes::vectors::PrimitiveVector;
-use datatypes::with_match_primitive_type_id;
-use snafu::{ensure, OptionExt};
-
-use crate::function::Function;
-
-#[derive(Clone, Debug, Default)]
-pub struct ClampFunction;
-
-const CLAMP_NAME: &str = "clamp";
-
-impl Function for ClampFunction {
-    fn name(&self) -> &str {
-        CLAMP_NAME
-    }
-
-    fn return_type(&self, input_types: &[ConcreteDataType]) -> Result<ConcreteDataType> {
-        // Type check is done by `signature`
-        Ok(input_types[0].clone())
-    }
-
-    fn signature(&self) -> Signature {
-        // input, min, max
-        Signature::uniform(3, ConcreteDataType::numerics(), Volatility::Immutable)
-    }
-
-    fn eval(
-        &self,
-        _func_ctx: crate::function::FunctionContext,
-        columns: &[VectorRef],
-    ) -> Result<VectorRef> {
-        ensure!(
-            columns.len() == 3,
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The length of the args is not correct, expect exactly 3, have: {}",
-                    columns.len()
-                ),
-            }
-        );
-        ensure!(
-            columns[0].data_type().is_numeric(),
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The first arg's type is not numeric, have: {}",
-                    columns[0].data_type()
-                ),
-            }
-        );
-        ensure!(
-            columns[0].data_type() == columns[1].data_type()
-                && columns[1].data_type() == columns[2].data_type(),
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "Arguments don't have identical types: {}, {}, {}",
-                    columns[0].data_type(),
-                    columns[1].data_type(),
-                    columns[2].data_type()
-                ),
-            }
-        );
-        ensure!(
-            columns[1].len() == 1 && columns[2].len() == 1,
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The second and third args should be scalar, have: {:?}, {:?}",
-                    columns[1], columns[2]
-                ),
-            }
-        );
-
-        with_match_primitive_type_id!(columns[0].data_type().logical_type_id(), |$S| {
-            let input_array = columns[0].to_arrow_array();
-            let input = input_array
-                    .as_any()
-                    .downcast_ref::<PrimitiveArray<<$S as LogicalPrimitiveType>::ArrowPrimitive>>()
-                    .unwrap();
-
-            let min = TryAsPrimitive::<$S>::try_as_primitive(&columns[1].get(0))
-                .with_context(|| {
-                    InvalidFuncArgsSnafu {
-                        err_msg: "The second arg should not be none",
-                    }
-                })?;
-            let max = TryAsPrimitive::<$S>::try_as_primitive(&columns[2].get(0))
-                .with_context(|| {
-                    InvalidFuncArgsSnafu {
-                        err_msg: "The third arg should not be none",
-                    }
-                })?;
-
-            // ensure min <= max
-            ensure!(
-                min <= max,
-                    InvalidFuncArgsSnafu {
-                        err_msg: format!(
-                        "The second arg should be less than or equal to the third arg, have: {:?}, {:?}",
-                        columns[1], columns[2]
-                    ),
-                }
-            );
-
-            clamp_impl::<$S, true, true>(input, min, max)
-        },{
-            unreachable!()
-        })
-    }
-}
-
-impl Display for ClampFunction {
-    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-        write!(f, "{}", CLAMP_NAME.to_ascii_uppercase())
-    }
-}
-
-fn clamp_impl<T: LogicalPrimitiveType, const CLAMP_MIN: bool, const CLAMP_MAX: bool>(
-    input: &PrimitiveArray<T::ArrowPrimitive>,
-    min: T::Native,
-    max: T::Native,
-) -> Result<VectorRef> {
-    common_telemetry::info!("[DEBUG] min {min:?}, max {max:?}");
-
-    let iter = ArrayIter::new(input);
-    let result = iter.map(|x| {
-        x.map(|x| {
-            if CLAMP_MIN && x < min {
-                min
-            } else if CLAMP_MAX && x > max {
-                max
-            } else {
-                x
-            }
-        })
-    });
-    let result = PrimitiveArray::<T::ArrowPrimitive>::from_iter(result);
-    Ok(Arc::new(PrimitiveVector::<T>::from(result)))
-}
-
-#[cfg(test)]
-mod test {
-
-    use std::sync::Arc;
-
-    use datatypes::prelude::ScalarVector;
-    use datatypes::vectors::{
-        ConstantVector, Float64Vector, Int64Vector, StringVector, UInt64Vector,
-    };
-
-    use super::*;
-    use crate::function::FunctionContext;
-
-    #[test]
-    fn clamp_i64() {
-        let inputs = [
-            (
-                vec![Some(-3), Some(-2), Some(-1), Some(0), Some(1), Some(2)],
-                -1,
-                10,
-                vec![Some(-1), Some(-1), Some(-1), Some(0), Some(1), Some(2)],
-            ),
-            (
-                vec![Some(-3), Some(-2), Some(-1), Some(0), Some(1), Some(2)],
-                0,
-                0,
-                vec![Some(0), Some(0), Some(0), Some(0), Some(0), Some(0)],
-            ),
-            (
-                vec![Some(-3), None, Some(-1), None, None, Some(2)],
-                -2,
-                1,
-                vec![Some(-2), None, Some(-1), None, None, Some(1)],
-            ),
-            (
-                vec![None, None, None, None, None],
-                0,
-                1,
-                vec![None, None, None, None, None],
-            ),
-        ];
-
-        let func = ClampFunction;
-        for (in_data, min, max, expected) in inputs {
-            let args = [
-                Arc::new(Int64Vector::from(in_data)) as _,
-                Arc::new(Int64Vector::from_vec(vec![min])) as _,
-                Arc::new(Int64Vector::from_vec(vec![max])) as _,
-            ];
-            let result = func
-                .eval(FunctionContext::default(), args.as_slice())
-                .unwrap();
-            let expected: VectorRef = Arc::new(Int64Vector::from(expected));
-            assert_eq!(expected, result);
-        }
-    }
-
-    #[test]
-    fn clamp_u64() {
-        let inputs = [
-            (
-                vec![Some(0), Some(1), Some(2), Some(3), Some(4), Some(5)],
-                1,
-                3,
-                vec![Some(1), Some(1), Some(2), Some(3), Some(3), Some(3)],
-            ),
-            (
-                vec![Some(0), Some(1), Some(2), Some(3), Some(4), Some(5)],
-                0,
-                0,
-                vec![Some(0), Some(0), Some(0), Some(0), Some(0), Some(0)],
-            ),
-            (
-                vec![Some(0), None, Some(2), None, None, Some(5)],
-                1,
-                3,
-                vec![Some(1), None, Some(2), None, None, Some(3)],
-            ),
-            (
-                vec![None, None, None, None, None],
-                0,
-                1,
-                vec![None, None, None, None, None],
-            ),
-        ];
-
-        let func = ClampFunction;
-        for (in_data, min, max, expected) in inputs {
-            let args = [
-                Arc::new(UInt64Vector::from(in_data)) as _,
-                Arc::new(UInt64Vector::from_vec(vec![min])) as _,
-                Arc::new(UInt64Vector::from_vec(vec![max])) as _,
-            ];
-            let result = func
-                .eval(FunctionContext::default(), args.as_slice())
-                .unwrap();
-            let expected: VectorRef = Arc::new(UInt64Vector::from(expected));
-            assert_eq!(expected, result);
-        }
-    }
-
-    #[test]
-    fn clamp_f64() {
-        let inputs = [
-            (
-                vec![Some(-3.0), Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)],
-                -1.0,
-                10.0,
-                vec![Some(-1.0), Some(-1.0), Some(-1.0), Some(0.0), Some(1.0)],
-            ),
-            (
-                vec![Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)],
-                0.0,
-                0.0,
-                vec![Some(0.0), Some(0.0), Some(0.0), Some(0.0)],
-            ),
-            (
-                vec![Some(-3.0), None, Some(-1.0), None, None, Some(2.0)],
-                -2.0,
-                1.0,
-                vec![Some(-2.0), None, Some(-1.0), None, None, Some(1.0)],
-            ),
-            (
-                vec![None, None, None, None, None],
-                0.0,
-                1.0,
-                vec![None, None, None, None, None],
-            ),
-        ];
-
-        let func = ClampFunction;
-        for (in_data, min, max, expected) in inputs {
-            let args = [
-                Arc::new(Float64Vector::from(in_data)) as _,
-                Arc::new(Float64Vector::from_vec(vec![min])) as _,
-                Arc::new(Float64Vector::from_vec(vec![max])) as _,
-            ];
-            let result = func
-                .eval(FunctionContext::default(), args.as_slice())
-                .unwrap();
-            let expected: VectorRef = Arc::new(Float64Vector::from(expected));
-            assert_eq!(expected, result);
-        }
-    }
-
-    #[test]
-    fn clamp_const_i32() {
-        let input = vec![Some(5)];
-        let min = 2;
-        let max = 4;
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(ConstantVector::new(Arc::new(Int64Vector::from(input)), 1)) as _,
-            Arc::new(Int64Vector::from_vec(vec![min])) as _,
-            Arc::new(Int64Vector::from_vec(vec![max])) as _,
-        ];
-        let result = func
-            .eval(FunctionContext::default(), args.as_slice())
-            .unwrap();
-        let expected: VectorRef = Arc::new(Int64Vector::from(vec![Some(4)]));
-        assert_eq!(expected, result);
-    }
-
-    #[test]
-    fn clamp_invalid_min_max() {
-        let input = vec![Some(-3.0), Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)];
-        let min = 10.0;
-        let max = -1.0;
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(Float64Vector::from(input)) as _,
-            Arc::new(Float64Vector::from_vec(vec![min])) as _,
-            Arc::new(Float64Vector::from_vec(vec![max])) as _,
-        ];
-        let result = func.eval(FunctionContext::default(), args.as_slice());
-        assert!(result.is_err());
-    }
-
-    #[test]
-    fn clamp_type_not_match() {
-        let input = vec![Some(-3.0), Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)];
-        let min = -1;
-        let max = 10;
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(Float64Vector::from(input)) as _,
-            Arc::new(Int64Vector::from_vec(vec![min])) as _,
-            Arc::new(UInt64Vector::from_vec(vec![max])) as _,
-        ];
-        let result = func.eval(FunctionContext::default(), args.as_slice());
-        assert!(result.is_err());
-    }
-
-    #[test]
-    fn clamp_min_is_not_scalar() {
-        let input = vec![Some(-3.0), Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)];
-        let min = -10.0;
-        let max = 1.0;
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(Float64Vector::from(input)) as _,
-            Arc::new(Float64Vector::from_vec(vec![min, min])) as _,
-            Arc::new(Float64Vector::from_vec(vec![max])) as _,
-        ];
-        let result = func.eval(FunctionContext::default(), args.as_slice());
-        assert!(result.is_err());
-    }
-
-    #[test]
-    fn clamp_no_max() {
-        let input = vec![Some(-3.0), Some(-2.0), Some(-1.0), Some(0.0), Some(1.0)];
-        let min = -10.0;
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(Float64Vector::from(input)) as _,
-            Arc::new(Float64Vector::from_vec(vec![min])) as _,
-        ];
-        let result = func.eval(FunctionContext::default(), args.as_slice());
-        assert!(result.is_err());
-    }
-
-    #[test]
-    fn clamp_on_string() {
-        let input = vec![Some("foo"), Some("foo"), Some("foo"), Some("foo")];
-
-        let func = ClampFunction;
-        let args = [
-            Arc::new(StringVector::from(input)) as _,
-            Arc::new(StringVector::from_vec(vec!["bar"])) as _,
-            Arc::new(StringVector::from_vec(vec!["baz"])) as _,
-        ];
-        let result = func.eval(FunctionContext::default(), args.as_slice());
-        assert!(result.is_err());
-    }
-}

src/common/function/src/scalars/math/modulo.rs

@@ -128,7 +128,7 @@ mod tests {
         ];
         let result = function.eval(FunctionContext::default(), &args).unwrap();
         assert_eq!(result.len(), 4);
-        for i in 0..4 {
+        for i in 0..3 {
             let p: i64 = (nums[i] % divs[i]) as i64;
             assert!(matches!(result.get(i), Value::Int64(v) if v == p));
         }
@@ -160,7 +160,7 @@ mod tests {
         ];
         let result = function.eval(FunctionContext::default(), &args).unwrap();
         assert_eq!(result.len(), 4);
-        for i in 0..4 {
+        for i in 0..3 {
             let p: u64 = (nums[i] % divs[i]) as u64;
             assert!(matches!(result.get(i), Value::UInt64(v) if v == p));
         }
@@ -192,7 +192,7 @@ mod tests {
         ];
         let result = function.eval(FunctionContext::default(), &args).unwrap();
         assert_eq!(result.len(), 4);
-        for i in 0..4 {
+        for i in 0..3 {
             let p: f64 = nums[i] % divs[i];
             assert!(matches!(result.get(i), Value::Float64(v) if v == p));
         }

src/common/function/src/scalars/timestamp.rs

@@ -14,11 +14,9 @@
 
 use std::sync::Arc;
 mod greatest;
-mod to_timezone;
 mod to_unixtime;
 
 use greatest::GreatestFunction;
-use to_timezone::ToTimezoneFunction;
 use to_unixtime::ToUnixtimeFunction;
 
 use crate::function_registry::FunctionRegistry;
@@ -27,7 +25,6 @@ pub(crate) struct TimestampFunction;
 
 impl TimestampFunction {
     pub fn register(registry: &FunctionRegistry) {
-        registry.register(Arc::new(ToTimezoneFunction));
         registry.register(Arc::new(ToUnixtimeFunction));
         registry.register(Arc::new(GreatestFunction));
     }

src/common/function/src/scalars/timestamp/to_timezone.rs

@@ -1,260 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt;
-use std::sync::Arc;
-
-use common_query::error::{InvalidFuncArgsSnafu, Result, UnsupportedInputDataTypeSnafu};
-use common_query::prelude::Signature;
-use common_time::{Timestamp, Timezone};
-use datatypes::data_type::ConcreteDataType;
-use datatypes::prelude::VectorRef;
-use datatypes::types::TimestampType;
-use datatypes::value::Value;
-use datatypes::vectors::{
-    StringVector, TimestampMicrosecondVector, TimestampMillisecondVector,
-    TimestampNanosecondVector, TimestampSecondVector, Vector,
-};
-use snafu::{ensure, OptionExt};
-
-use crate::function::{Function, FunctionContext};
-use crate::helper;
-
-#[derive(Clone, Debug, Default)]
-pub struct ToTimezoneFunction;
-
-const NAME: &str = "to_timezone";
-
-fn convert_to_timezone(arg: &str) -> Option<Timezone> {
-    Timezone::from_tz_string(arg).ok()
-}
-
-fn convert_to_timestamp(arg: &Value) -> Option<Timestamp> {
-    match arg {
-        Value::Timestamp(ts) => Some(*ts),
-        _ => None,
-    }
-}
-
-impl fmt::Display for ToTimezoneFunction {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        write!(f, "TO_TIMEZONE")
-    }
-}
-
-impl Function for ToTimezoneFunction {
-    fn name(&self) -> &str {
-        NAME
-    }
-
-    fn return_type(&self, input_types: &[ConcreteDataType]) -> Result<ConcreteDataType> {
-        // type checked by signature - MUST BE timestamp
-        Ok(input_types[0].clone())
-    }
-
-    fn signature(&self) -> Signature {
-        helper::one_of_sigs2(
-            vec![
-                ConcreteDataType::timestamp_second_datatype(),
-                ConcreteDataType::timestamp_millisecond_datatype(),
-                ConcreteDataType::timestamp_microsecond_datatype(),
-                ConcreteDataType::timestamp_nanosecond_datatype(),
-            ],
-            vec![ConcreteDataType::string_datatype()],
-        )
-    }
-
-    fn eval(&self, _ctx: FunctionContext, columns: &[VectorRef]) -> Result<VectorRef> {
-        ensure!(
-            columns.len() == 2,
-            InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The length of the args is not correct, expect exactly 2, have: {}",
-                    columns.len()
-                ),
-            }
-        );
-
-        // TODO: maybe support epoch timestamp? https://github.com/GreptimeTeam/greptimedb/issues/3477
-        let ts = columns[0].data_type().as_timestamp().with_context(|| {
-            UnsupportedInputDataTypeSnafu {
-                function: NAME,
-                datatypes: columns.iter().map(|c| c.data_type()).collect::<Vec<_>>(),
-            }
-        })?;
-        let array = columns[0].to_arrow_array();
-        let times = match ts {
-            TimestampType::Second(_) => {
-                let vector = TimestampSecondVector::try_from_arrow_array(array).unwrap();
-                (0..vector.len())
-                    .map(|i| convert_to_timestamp(&vector.get(i)))
-                    .collect::<Vec<_>>()
-            }
-            TimestampType::Millisecond(_) => {
-                let vector = TimestampMillisecondVector::try_from_arrow_array(array).unwrap();
-                (0..vector.len())
-                    .map(|i| convert_to_timestamp(&vector.get(i)))
-                    .collect::<Vec<_>>()
-            }
-            TimestampType::Microsecond(_) => {
-                let vector = TimestampMicrosecondVector::try_from_arrow_array(array).unwrap();
-                (0..vector.len())
-                    .map(|i| convert_to_timestamp(&vector.get(i)))
-                    .collect::<Vec<_>>()
-            }
-            TimestampType::Nanosecond(_) => {
-                let vector = TimestampNanosecondVector::try_from_arrow_array(array).unwrap();
-                (0..vector.len())
-                    .map(|i| convert_to_timestamp(&vector.get(i)))
-                    .collect::<Vec<_>>()
-            }
-        };
-
-        let tzs = {
-            let array = columns[1].to_arrow_array();
-            let vector = StringVector::try_from_arrow_array(&array)
-                .ok()
-                .with_context(|| UnsupportedInputDataTypeSnafu {
-                    function: NAME,
-                    datatypes: columns.iter().map(|c| c.data_type()).collect::<Vec<_>>(),
-                })?;
-            (0..vector.len())
-                .map(|i| convert_to_timezone(&vector.get(i).to_string()))
-                .collect::<Vec<_>>()
-        };
-
-        let result = times
-            .iter()
-            .zip(tzs.iter())
-            .map(|(time, tz)| match (time, tz) {
-                (Some(time), _) => Some(time.to_timezone_aware_string(tz.as_ref())),
-                _ => None,
-            })
-            .collect::<Vec<Option<String>>>();
-        Ok(Arc::new(StringVector::from(result)))
-    }
-}
-
-#[cfg(test)]
-mod tests {
-
-    use datatypes::scalars::ScalarVector;
-    use datatypes::timestamp::{
-        TimestampMicrosecond, TimestampMillisecond, TimestampNanosecond, TimestampSecond,
-    };
-    use datatypes::vectors::StringVector;
-
-    use super::*;
-
-    #[test]
-    fn test_timestamp_to_timezone() {
-        let f = ToTimezoneFunction;
-        assert_eq!("to_timezone", f.name());
-
-        let results = vec![
-            Some("1969-12-31 19:00:01"),
-            None,
-            Some("1970-01-01 03:00:01"),
-            None,
-        ];
-        let times: Vec<Option<TimestampSecond>> = vec![
-            Some(TimestampSecond::new(1)),
-            None,
-            Some(TimestampSecond::new(1)),
-            None,
-        ];
-        let ts_vector: TimestampSecondVector =
-            TimestampSecondVector::from_owned_iterator(times.into_iter());
-        let tzs = vec![Some("America/New_York"), None, Some("Europe/Moscow"), None];
-        let args: Vec<VectorRef> = vec![
-            Arc::new(ts_vector),
-            Arc::new(StringVector::from(tzs.clone())),
-        ];
-        let vector = f.eval(FunctionContext::default(), &args).unwrap();
-        assert_eq!(4, vector.len());
-        let expect_times: VectorRef = Arc::new(StringVector::from(results));
-        assert_eq!(expect_times, vector);
-
-        let results = vec![
-            Some("1969-12-31 19:00:00.001"),
-            None,
-            Some("1970-01-01 03:00:00.001"),
-            None,
-        ];
-        let times: Vec<Option<TimestampMillisecond>> = vec![
-            Some(TimestampMillisecond::new(1)),
-            None,
-            Some(TimestampMillisecond::new(1)),
-            None,
-        ];
-        let ts_vector: TimestampMillisecondVector =
-            TimestampMillisecondVector::from_owned_iterator(times.into_iter());
-        let args: Vec<VectorRef> = vec![
-            Arc::new(ts_vector),
-            Arc::new(StringVector::from(tzs.clone())),
-        ];
-        let vector = f.eval(FunctionContext::default(), &args).unwrap();
-        assert_eq!(4, vector.len());
-        let expect_times: VectorRef = Arc::new(StringVector::from(results));
-        assert_eq!(expect_times, vector);
-
-        let results = vec![
-            Some("1969-12-31 19:00:00.000001"),
-            None,
-            Some("1970-01-01 03:00:00.000001"),
-            None,
-        ];
-        let times: Vec<Option<TimestampMicrosecond>> = vec![
-            Some(TimestampMicrosecond::new(1)),
-            None,
-            Some(TimestampMicrosecond::new(1)),
-            None,
-        ];
-        let ts_vector: TimestampMicrosecondVector =
-            TimestampMicrosecondVector::from_owned_iterator(times.into_iter());
-
-        let args: Vec<VectorRef> = vec![
-            Arc::new(ts_vector),
-            Arc::new(StringVector::from(tzs.clone())),
-        ];
-        let vector = f.eval(FunctionContext::default(), &args).unwrap();
-        assert_eq!(4, vector.len());
-        let expect_times: VectorRef = Arc::new(StringVector::from(results));
-        assert_eq!(expect_times, vector);
-
-        let results = vec![
-            Some("1969-12-31 19:00:00.000000001"),
-            None,
-            Some("1970-01-01 03:00:00.000000001"),
-            None,
-        ];
-        let times: Vec<Option<TimestampNanosecond>> = vec![
-            Some(TimestampNanosecond::new(1)),
-            None,
-            Some(TimestampNanosecond::new(1)),
-            None,
-        ];
-        let ts_vector: TimestampNanosecondVector =
-            TimestampNanosecondVector::from_owned_iterator(times.into_iter());
-
-        let args: Vec<VectorRef> = vec![
-            Arc::new(ts_vector),
-            Arc::new(StringVector::from(tzs.clone())),
-        ];
-        let vector = f.eval(FunctionContext::default(), &args).unwrap();
-        assert_eq!(4, vector.len());
-        let expect_times: VectorRef = Arc::new(StringVector::from(results));
-        assert_eq!(expect_times, vector);
-    }
-}

src/common/function/src/state.rs

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-use crate::handlers::{ProcedureServiceHandlerRef, TableMutationHandlerRef};
+use crate::handlers::{MetaServiceHandlerRef, TableMutationHandlerRef};
 
 /// Shared state for SQL functions.
 /// The handlers in state may be `None` in cli command-line or test cases.
@@ -20,105 +20,6 @@ use crate::handlers::{ProcedureServiceHandlerRef, TableMutationHandlerRef};
 pub struct FunctionState {
     // The table mutation handler
     pub table_mutation_handler: Option<TableMutationHandlerRef>,
-    // The procedure service handler
-    pub procedure_service_handler: Option<ProcedureServiceHandlerRef>,
-}
-
-impl FunctionState {
-    /// Create a mock [`FunctionState`] for test.
-    #[cfg(any(test, feature = "testing"))]
-    pub fn mock() -> Self {
-        use std::sync::Arc;
-
-        use api::v1::meta::ProcedureStatus;
-        use async_trait::async_trait;
-        use common_base::AffectedRows;
-        use common_meta::rpc::procedure::{MigrateRegionRequest, ProcedureStateResponse};
-        use common_query::error::Result;
-        use common_query::Output;
-        use session::context::QueryContextRef;
-        use store_api::storage::RegionId;
-        use table::requests::{
-            CompactTableRequest, DeleteRequest, FlushTableRequest, InsertRequest,
-        };
-
-        use crate::handlers::{ProcedureServiceHandler, TableMutationHandler};
-        struct MockProcedureServiceHandler;
-        struct MockTableMutationHandler;
-        const ROWS: usize = 42;
-
-        #[async_trait]
-        impl ProcedureServiceHandler for MockProcedureServiceHandler {
-            async fn migrate_region(
-                &self,
-                _request: MigrateRegionRequest,
-            ) -> Result<Option<String>> {
-                Ok(Some("test_pid".to_string()))
-            }
-
-            async fn query_procedure_state(&self, _pid: &str) -> Result<ProcedureStateResponse> {
-                Ok(ProcedureStateResponse {
-                    status: ProcedureStatus::Done.into(),
-                    error: "OK".to_string(),
-                    ..Default::default()
-                })
-            }
-        }
-
-        #[async_trait]
-        impl TableMutationHandler for MockTableMutationHandler {
-            async fn insert(
-                &self,
-                _request: InsertRequest,
-                _ctx: QueryContextRef,
-            ) -> Result<Output> {
-                Ok(Output::new_with_affected_rows(ROWS))
-            }
-
-            async fn delete(
-                &self,
-                _request: DeleteRequest,
-                _ctx: QueryContextRef,
-            ) -> Result<AffectedRows> {
-                Ok(ROWS)
-            }
-
-            async fn flush(
-                &self,
-                _request: FlushTableRequest,
-                _ctx: QueryContextRef,
-            ) -> Result<AffectedRows> {
-                Ok(ROWS)
-            }
-
-            async fn compact(
-                &self,
-                _request: CompactTableRequest,
-                _ctx: QueryContextRef,
-            ) -> Result<AffectedRows> {
-                Ok(ROWS)
-            }
-
-            async fn flush_region(
-                &self,
-                _region_id: RegionId,
-                _ctx: QueryContextRef,
-            ) -> Result<AffectedRows> {
-                Ok(ROWS)
-            }
-
-            async fn compact_region(
-                &self,
-                _region_id: RegionId,
-                _ctx: QueryContextRef,
-            ) -> Result<AffectedRows> {
-                Ok(ROWS)
-            }
-        }
-
-        Self {
-            table_mutation_handler: Some(Arc::new(MockTableMutationHandler)),
-            procedure_service_handler: Some(Arc::new(MockProcedureServiceHandler)),
-        }
-    }
+    // The meta service handler
+    pub meta_service_handler: Option<MetaServiceHandlerRef>,
 }

src/common/function/src/system.rs

@@ -14,7 +14,6 @@
 
 mod build;
 mod database;
-mod procedure_state;
 mod timezone;
 mod version;
 
@@ -22,7 +21,6 @@ use std::sync::Arc;
 
 use build::BuildFunction;
 use database::DatabaseFunction;
-use procedure_state::ProcedureStateFunction;
 use timezone::TimezoneFunction;
 use version::VersionFunction;
 
@@ -36,6 +34,5 @@ impl SystemFunction {
         registry.register(Arc::new(VersionFunction));
         registry.register(Arc::new(DatabaseFunction));
         registry.register(Arc::new(TimezoneFunction));
-        registry.register(Arc::new(ProcedureStateFunction));
     }
 }

src/common/function/src/system/build.rs

@@ -22,7 +22,7 @@ use datatypes::vectors::{StringVector, VectorRef};
 
 use crate::function::{Function, FunctionContext};
 
-/// Generates build information
+/// Generates build information  
 #[derive(Clone, Debug, Default)]
 pub struct BuildFunction;
 
@@ -42,7 +42,11 @@ impl Function for BuildFunction {
     }
 
     fn signature(&self) -> Signature {
-        Signature::uniform(0, vec![], Volatility::Immutable)
+        Signature::uniform(
+            0,
+            vec![ConcreteDataType::string_datatype()],
+            Volatility::Immutable,
+        )
     }
 
     fn eval(&self, _func_ctx: FunctionContext, _columns: &[VectorRef]) -> Result<VectorRef> {
@@ -71,7 +75,7 @@ mod tests {
                          Signature {
                              type_signature: TypeSignature::Uniform(0, valid_types),
                              volatility: Volatility::Immutable
-                         } if  valid_types.is_empty()
+                         } if  valid_types == vec![ConcreteDataType::string_datatype()]
         ));
         let build_info = common_version::build_info().to_string();
         let vector = build.eval(FunctionContext::default(), &[]).unwrap();

src/common/function/src/system/procedure_state.rs

@@ -1,159 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt;
-
-use api::v1::meta::ProcedureStatus;
-use common_macro::admin_fn;
-use common_meta::rpc::procedure::ProcedureStateResponse;
-use common_query::error::Error::ThreadJoin;
-use common_query::error::{
-    InvalidFuncArgsSnafu, MissingProcedureServiceHandlerSnafu, Result,
-    UnsupportedInputDataTypeSnafu,
-};
-use common_query::prelude::{Signature, Volatility};
-use common_telemetry::error;
-use datatypes::prelude::*;
-use datatypes::vectors::VectorRef;
-use serde::Serialize;
-use session::context::QueryContextRef;
-use snafu::{ensure, Location, OptionExt};
-
-use crate::ensure_greptime;
-use crate::function::{Function, FunctionContext};
-use crate::handlers::ProcedureServiceHandlerRef;
-
-#[derive(Serialize)]
-struct ProcedureStateJson {
-    status: String,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    error: Option<String>,
-}
-
-/// A function to query procedure state by its id.
-/// Such as `procedure_state(pid)`.
-#[admin_fn(
-    name = "ProcedureStateFunction",
-    display_name = "procedure_state",
-    sig_fn = "signature",
-    ret = "string"
-)]
-pub(crate) async fn procedure_state(
-    procedure_service_handler: &ProcedureServiceHandlerRef,
-    _ctx: &QueryContextRef,
-    params: &[ValueRef<'_>],
-) -> Result<Value> {
-    ensure!(
-        params.len() == 1,
-        InvalidFuncArgsSnafu {
-            err_msg: format!(
-                "The length of the args is not correct, expect 1, have: {}",
-                params.len()
-            ),
-        }
-    );
-
-    let ValueRef::String(pid) = params[0] else {
-        return UnsupportedInputDataTypeSnafu {
-            function: "procedure_state",
-            datatypes: params.iter().map(|v| v.data_type()).collect::<Vec<_>>(),
-        }
-        .fail();
-    };
-
-    let ProcedureStateResponse { status, error, .. } =
-        procedure_service_handler.query_procedure_state(pid).await?;
-    let status = ProcedureStatus::try_from(status)
-        .map(|v| v.as_str_name())
-        .unwrap_or("Unknown");
-
-    let state = ProcedureStateJson {
-        status: status.to_string(),
-        error: if error.is_empty() { None } else { Some(error) },
-    };
-    let json = serde_json::to_string(&state).unwrap_or_default();
-
-    Ok(Value::from(json))
-}
-
-fn signature() -> Signature {
-    Signature::uniform(
-        1,
-        vec![ConcreteDataType::string_datatype()],
-        Volatility::Immutable,
-    )
-}
-
-#[cfg(test)]
-mod tests {
-    use std::sync::Arc;
-
-    use common_query::prelude::TypeSignature;
-    use datatypes::vectors::StringVector;
-
-    use super::*;
-
-    #[test]
-    fn test_procedure_state_misc() {
-        let f = ProcedureStateFunction;
-        assert_eq!("procedure_state", f.name());
-        assert_eq!(
-            ConcreteDataType::string_datatype(),
-            f.return_type(&[]).unwrap()
-        );
-        assert!(matches!(f.signature(),
-                         Signature {
-                             type_signature: TypeSignature::Uniform(1, valid_types),
-                             volatility: Volatility::Immutable
-                         } if valid_types == vec![ConcreteDataType::string_datatype()]
-        ));
-    }
-
-    #[test]
-    fn test_missing_procedure_service() {
-        let f = ProcedureStateFunction;
-
-        let args = vec!["pid"];
-
-        let args = args
-            .into_iter()
-            .map(|arg| Arc::new(StringVector::from_slice(&[arg])) as _)
-            .collect::<Vec<_>>();
-
-        let result = f.eval(FunctionContext::default(), &args).unwrap_err();
-        assert_eq!(
-            "Missing ProcedureServiceHandler, not expected",
-            result.to_string()
-        );
-    }
-
-    #[test]
-    fn test_procedure_state() {
-        let f = ProcedureStateFunction;
-
-        let args = vec!["pid"];
-
-        let args = args
-            .into_iter()
-            .map(|arg| Arc::new(StringVector::from_slice(&[arg])) as _)
-            .collect::<Vec<_>>();
-
-        let result = f.eval(FunctionContext::mock(), &args).unwrap();
-
-        let expect: VectorRef = Arc::new(StringVector::from(vec![
-            "{\"status\":\"Done\",\"error\":\"OK\"}",
-        ]));
-        assert_eq!(expect, result);
-    }
-}

src/common/function/src/table.rs

@@ -12,14 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-mod flush_compact_region;
-mod flush_compact_table;
 mod migrate_region;
 
 use std::sync::Arc;
 
-use flush_compact_region::{CompactRegionFunction, FlushRegionFunction};
-use flush_compact_table::{CompactTableFunction, FlushTableFunction};
 use migrate_region::MigrateRegionFunction;
 
 use crate::function_registry::FunctionRegistry;
@@ -31,9 +27,5 @@ impl TableFunction {
     /// Register all table functions to [`FunctionRegistry`].
     pub fn register(registry: &FunctionRegistry) {
         registry.register(Arc::new(MigrateRegionFunction));
-        registry.register(Arc::new(FlushRegionFunction));
-        registry.register(Arc::new(CompactRegionFunction));
-        registry.register(Arc::new(FlushTableFunction));
-        registry.register(Arc::new(CompactTableFunction));
     }
 }

src/common/function/src/table/flush_compact_region.rs

@@ -1,148 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt;
-
-use common_macro::admin_fn;
-use common_query::error::Error::ThreadJoin;
-use common_query::error::{
-    InvalidFuncArgsSnafu, MissingTableMutationHandlerSnafu, Result, UnsupportedInputDataTypeSnafu,
-};
-use common_query::prelude::{Signature, Volatility};
-use common_telemetry::error;
-use datatypes::prelude::*;
-use datatypes::vectors::VectorRef;
-use session::context::QueryContextRef;
-use snafu::{ensure, Location, OptionExt};
-use store_api::storage::RegionId;
-
-use crate::ensure_greptime;
-use crate::function::{Function, FunctionContext};
-use crate::handlers::TableMutationHandlerRef;
-use crate::helper::cast_u64;
-
-macro_rules! define_region_function {
-    ($name: expr, $display_name_str: expr, $display_name: ident) => {
-        /// A function to $display_name
-        #[admin_fn(name = $name, display_name = $display_name_str, sig_fn = "signature", ret = "uint64")]
-        pub(crate) async fn $display_name(
-            table_mutation_handler: &TableMutationHandlerRef,
-            query_ctx: &QueryContextRef,
-            params: &[ValueRef<'_>],
-        ) -> Result<Value> {
-            ensure!(
-                params.len() == 1,
-                InvalidFuncArgsSnafu {
-                    err_msg: format!(
-                        "The length of the args is not correct, expect 1, have: {}",
-                        params.len()
-                    ),
-                }
-            );
-
-            let Some(region_id) = cast_u64(&params[0])? else {
-                return UnsupportedInputDataTypeSnafu {
-                    function: $display_name_str,
-                    datatypes: params.iter().map(|v| v.data_type()).collect::<Vec<_>>(),
-                }
-                .fail();
-            };
-
-            let affected_rows = table_mutation_handler
-                .$display_name(RegionId::from_u64(region_id), query_ctx.clone())
-                .await?;
-
-            Ok(Value::from(affected_rows as u64))
-        }
-    };
-}
-
-define_region_function!("FlushRegionFunction", "flush_region", flush_region);
-
-define_region_function!("CompactRegionFunction", "compact_region", compact_region);
-
-fn signature() -> Signature {
-    Signature::uniform(1, ConcreteDataType::numerics(), Volatility::Immutable)
-}
-
-#[cfg(test)]
-mod tests {
-    use std::sync::Arc;
-
-    use common_query::prelude::TypeSignature;
-    use datatypes::vectors::UInt64Vector;
-
-    use super::*;
-
-    macro_rules! define_region_function_test {
-        ($name: ident, $func: ident) => {
-            paste::paste! {
-                #[test]
-                fn [<test_ $name _misc>]() {
-                    let f = $func;
-                    assert_eq!(stringify!($name), f.name());
-                    assert_eq!(
-                        ConcreteDataType::uint64_datatype(),
-                        f.return_type(&[]).unwrap()
-                    );
-                    assert!(matches!(f.signature(),
-                                     Signature {
-                                         type_signature: TypeSignature::Uniform(1, valid_types),
-                                         volatility: Volatility::Immutable
-                                     } if valid_types == ConcreteDataType::numerics()));
-                }
-
-                #[test]
-                fn [<test_ $name _missing_table_mutation>]() {
-                    let f = $func;
-
-                    let args = vec![99];
-
-                    let args = args
-                        .into_iter()
-                        .map(|arg| Arc::new(UInt64Vector::from_slice([arg])) as _)
-                        .collect::<Vec<_>>();
-
-                    let result = f.eval(FunctionContext::default(), &args).unwrap_err();
-                    assert_eq!(
-                        "Missing TableMutationHandler, not expected",
-                        result.to_string()
-                    );
-                }
-
-                #[test]
-                fn [<test_ $name>]() {
-                    let f = $func;
-
-
-                    let args = vec![99];
-
-                    let args = args
-                        .into_iter()
-                        .map(|arg| Arc::new(UInt64Vector::from_slice([arg])) as _)
-                        .collect::<Vec<_>>();
-
-                    let result = f.eval(FunctionContext::mock(), &args).unwrap();
-
-                    let expect: VectorRef = Arc::new(UInt64Vector::from_slice([42]));
-                    assert_eq!(expect, result);
-                }
-            }
-        };
-    }
-
-    define_region_function_test!(flush_region, FlushRegionFunction);
-
-    define_region_function_test!(compact_region, CompactRegionFunction);
-}

src/common/function/src/table/flush_compact_table.rs

@@ -1,178 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::fmt;
-
-use common_error::ext::BoxedError;
-use common_macro::admin_fn;
-use common_query::error::Error::ThreadJoin;
-use common_query::error::{
-    InvalidFuncArgsSnafu, MissingTableMutationHandlerSnafu, Result, TableMutationSnafu,
-    UnsupportedInputDataTypeSnafu,
-};
-use common_query::prelude::{Signature, Volatility};
-use common_telemetry::error;
-use datatypes::prelude::*;
-use datatypes::vectors::VectorRef;
-use session::context::QueryContextRef;
-use session::table_name::table_name_to_full_name;
-use snafu::{ensure, Location, OptionExt, ResultExt};
-use table::requests::{CompactTableRequest, FlushTableRequest};
-
-use crate::ensure_greptime;
-use crate::function::{Function, FunctionContext};
-use crate::handlers::TableMutationHandlerRef;
-
-macro_rules! define_table_function {
-    ($name: expr, $display_name_str: expr, $display_name: ident, $func: ident, $request: ident) => {
-        /// A function to $func table, such as `$display_name(table_name)`.
-        #[admin_fn(name = $name, display_name = $display_name_str, sig_fn = "signature", ret = "uint64")]
-        pub(crate) async fn $display_name(
-            table_mutation_handler: &TableMutationHandlerRef,
-            query_ctx: &QueryContextRef,
-            params: &[ValueRef<'_>],
-        ) -> Result<Value> {
-            ensure!(
-                params.len() == 1,
-                InvalidFuncArgsSnafu {
-                    err_msg: format!(
-                        "The length of the args is not correct, expect 1, have: {}",
-                        params.len()
-                    ),
-                }
-            );
-
-            let ValueRef::String(table_name) = params[0] else {
-                return UnsupportedInputDataTypeSnafu {
-                    function: $display_name_str,
-                    datatypes: params.iter().map(|v| v.data_type()).collect::<Vec<_>>(),
-                }
-                .fail();
-            };
-
-            let (catalog_name, schema_name, table_name) =
-                table_name_to_full_name(table_name, &query_ctx)
-                    .map_err(BoxedError::new)
-                    .context(TableMutationSnafu)?;
-
-            let affected_rows = table_mutation_handler
-                .$func(
-                    $request {
-                        catalog_name,
-                        schema_name,
-                        table_name,
-                    },
-                    query_ctx.clone(),
-                )
-                .await?;
-
-            Ok(Value::from(affected_rows as u64))
-        }
-    };
-}
-
-define_table_function!(
-    "FlushTableFunction",
-    "flush_table",
-    flush_table,
-    flush,
-    FlushTableRequest
-);
-
-define_table_function!(
-    "CompactTableFunction",
-    "compact_table",
-    compact_table,
-    compact,
-    CompactTableRequest
-);
-
-fn signature() -> Signature {
-    Signature::uniform(
-        1,
-        vec![ConcreteDataType::string_datatype()],
-        Volatility::Immutable,
-    )
-}
-
-#[cfg(test)]
-mod tests {
-    use std::sync::Arc;
-
-    use common_query::prelude::TypeSignature;
-    use datatypes::vectors::{StringVector, UInt64Vector};
-
-    use super::*;
-
-    macro_rules! define_table_function_test {
-        ($name: ident, $func: ident) => {
-            paste::paste!{
-                #[test]
-                fn [<test_ $name _misc>]() {
-                    let f = $func;
-                    assert_eq!(stringify!($name), f.name());
-                    assert_eq!(
-                        ConcreteDataType::uint64_datatype(),
-                        f.return_type(&[]).unwrap()
-                    );
-                    assert!(matches!(f.signature(),
-                                     Signature {
-                                         type_signature: TypeSignature::Uniform(1, valid_types),
-                                         volatility: Volatility::Immutable
-                                     } if valid_types == vec![ConcreteDataType::string_datatype()]));
-                }
-
-                #[test]
-                fn [<test_ $name _missing_table_mutation>]() {
-                    let f = $func;
-
-                    let args = vec!["test"];
-
-                    let args = args
-                        .into_iter()
-                        .map(|arg| Arc::new(StringVector::from(vec![arg])) as _)
-                        .collect::<Vec<_>>();
-
-                    let result = f.eval(FunctionContext::default(), &args).unwrap_err();
-                    assert_eq!(
-                        "Missing TableMutationHandler, not expected",
-                        result.to_string()
-                    );
-                }
-
-                #[test]
-                fn [<test_ $name>]() {
-                    let f = $func;
-
-
-                    let args = vec!["test"];
-
-                    let args = args
-                        .into_iter()
-                        .map(|arg| Arc::new(StringVector::from(vec![arg])) as _)
-                        .collect::<Vec<_>>();
-
-                    let result = f.eval(FunctionContext::mock(), &args).unwrap();
-
-                    let expect: VectorRef = Arc::new(UInt64Vector::from_slice([42]));
-                    assert_eq!(expect, result);
-                }
-            }
-        }
-    }
-
-    define_table_function_test!(flush_table, FlushTableFunction);
-
-    define_table_function_test!(compact_table, CompactTableFunction);
-}

src/common/function/src/table/migrate_region.rs

@@ -15,25 +15,18 @@
 use std::fmt::{self};
 use std::time::Duration;
 
-use common_macro::admin_fn;
-use common_meta::rpc::procedure::MigrateRegionRequest;
 use common_query::error::Error::ThreadJoin;
-use common_query::error::{InvalidFuncArgsSnafu, MissingProcedureServiceHandlerSnafu, Result};
+use common_query::error::{
+    InvalidFuncArgsSnafu, InvalidInputTypeSnafu, MissingTableMutationHandlerSnafu, Result,
+};
 use common_query::prelude::{Signature, TypeSignature, Volatility};
 use common_telemetry::logging::error;
-use datatypes::data_type::DataType;
-use datatypes::prelude::ConcreteDataType;
-use datatypes::value::{Value, ValueRef};
-use datatypes::vectors::VectorRef;
-use session::context::QueryContextRef;
-use snafu::{Location, OptionExt};
+use datatypes::prelude::{ConcreteDataType, MutableVector, ScalarVectorBuilder};
+use datatypes::value::Value;
+use datatypes::vectors::{StringVectorBuilder, VectorRef};
+use snafu::{Location, OptionExt, ResultExt};
 
-use crate::ensure_greptime;
 use crate::function::{Function, FunctionContext};
-use crate::handlers::ProcedureServiceHandlerRef;
-use crate::helper::cast_u64;
-
-const DEFAULT_REPLAY_TIMEOUT_SECS: u64 = 10;
 
 /// A function to migrate a region from source peer to target peer.
 /// Returns the submitted procedure id if success. Only available in cluster mode.
@@ -45,140 +38,138 @@ const DEFAULT_REPLAY_TIMEOUT_SECS: u64 = 10;
 /// - `region_id`:  the region id
 /// - `from_peer`:  the source peer id
 /// - `to_peer`:  the target peer id
-#[admin_fn(
-    name = "MigrateRegionFunction",
-    display_name = "migrate_region",
-    sig_fn = "signature",
-    ret = "string"
-)]
-pub(crate) async fn migrate_region(
-    procedure_service_handler: &ProcedureServiceHandlerRef,
-    _ctx: &QueryContextRef,
-    params: &[ValueRef<'_>],
-) -> Result<Value> {
-    let (region_id, from_peer, to_peer, replay_timeout) = match params.len() {
-        3 => {
-            let region_id = cast_u64(&params[0])?;
-            let from_peer = cast_u64(&params[1])?;
-            let to_peer = cast_u64(&params[2])?;
+#[derive(Clone, Debug, Default)]
+pub struct MigrateRegionFunction;
 
-            (
-                region_id,
-                from_peer,
-                to_peer,
-                Some(DEFAULT_REPLAY_TIMEOUT_SECS),
-            )
-        }
+const NAME: &str = "migrate_region";
+const DEFAULT_REPLAY_TIMEOUT_SECS: u64 = 10;
 
-        4 => {
-            let region_id = cast_u64(&params[0])?;
-            let from_peer = cast_u64(&params[1])?;
-            let to_peer = cast_u64(&params[2])?;
-            let replay_timeout = cast_u64(&params[3])?;
+fn cast_u64_vector(vector: &VectorRef) -> Result<VectorRef> {
+    vector
+        .cast(&ConcreteDataType::uint64_datatype())
+        .context(InvalidInputTypeSnafu {
+            err_msg: format!(
+                "Failed to cast input into uint64, actual type: {:#?}",
+                vector.data_type(),
+            ),
+        })
+}
 
-            (region_id, from_peer, to_peer, replay_timeout)
-        }
+impl Function for MigrateRegionFunction {
+    fn name(&self) -> &str {
+        NAME
+    }
 
-        size => {
-            return InvalidFuncArgsSnafu {
-                err_msg: format!(
-                    "The length of the args is not correct, expect exactly 3 or 4, have: {}",
-                    size
-                ),
+    fn return_type(&self, _input_types: &[ConcreteDataType]) -> Result<ConcreteDataType> {
+        Ok(ConcreteDataType::string_datatype())
+    }
+
+    fn signature(&self) -> Signature {
+        Signature::one_of(
+            vec![
+                // migrate_region(region_id, from_peer, to_peer)
+                TypeSignature::Uniform(3, ConcreteDataType::numerics()),
+                // migrate_region(region_id, from_peer, to_peer, timeout(secs))
+                TypeSignature::Uniform(4, ConcreteDataType::numerics()),
+            ],
+            Volatility::Immutable,
+        )
+    }
+
+    fn eval(&self, func_ctx: FunctionContext, columns: &[VectorRef]) -> Result<VectorRef> {
+        let (region_ids, from_peers, to_peers, replay_timeouts) = match columns.len() {
+            3 => {
+                let region_ids = cast_u64_vector(&columns[0])?;
+                let from_peers = cast_u64_vector(&columns[1])?;
+                let to_peers = cast_u64_vector(&columns[2])?;
+
+                (region_ids, from_peers, to_peers, None)
             }
-            .fail();
-        }
-    };
 
-    match (region_id, from_peer, to_peer, replay_timeout) {
-        (Some(region_id), Some(from_peer), Some(to_peer), Some(replay_timeout)) => {
-            let pid = procedure_service_handler
-                .migrate_region(MigrateRegionRequest {
-                    region_id,
-                    from_peer,
-                    to_peer,
-                    replay_timeout: Duration::from_secs(replay_timeout),
-                })
-                .await?;
+            4 => {
+                let region_ids = cast_u64_vector(&columns[0])?;
+                let from_peers = cast_u64_vector(&columns[1])?;
+                let to_peers = cast_u64_vector(&columns[2])?;
+                let replay_timeouts = cast_u64_vector(&columns[3])?;
 
-            match pid {
-                Some(pid) => Ok(Value::from(pid)),
-                None => Ok(Value::Null),
+                (region_ids, from_peers, to_peers, Some(replay_timeouts))
             }
-        }
 
-        _ => Ok(Value::Null),
+            size => {
+                return InvalidFuncArgsSnafu {
+                    err_msg: format!(
+                        "The length of the args is not correct, expect exactly 3 or 4, have: {}",
+                        size
+                    ),
+                }
+                .fail();
+            }
+        };
+
+        std::thread::spawn(move || {
+            let len = region_ids.len();
+            let mut results = StringVectorBuilder::with_capacity(len);
+
+            for index in 0..len {
+                let region_id = region_ids.get(index);
+                let from_peer = from_peers.get(index);
+                let to_peer = to_peers.get(index);
+                let replay_timeout = match &replay_timeouts {
+                    Some(replay_timeouts) => replay_timeouts.get(index),
+                    None => Value::UInt64(DEFAULT_REPLAY_TIMEOUT_SECS),
+                };
+
+                match (region_id, from_peer, to_peer, replay_timeout) {
+                    (
+                        Value::UInt64(region_id),
+                        Value::UInt64(from_peer),
+                        Value::UInt64(to_peer),
+                        Value::UInt64(replay_timeout),
+                    ) => {
+                        let func_ctx = func_ctx.clone();
+
+                        let pid = common_runtime::block_on_read(async move {
+                            func_ctx
+                                .state
+                                .table_mutation_handler
+                                .as_ref()
+                                .context(MissingTableMutationHandlerSnafu)?
+                                .migrate_region(
+                                    region_id,
+                                    from_peer,
+                                    to_peer,
+                                    Duration::from_secs(replay_timeout),
+                                )
+                                .await
+                        })?;
+
+                        results.push(Some(&pid));
+                    }
+                    _ => {
+                        results.push(None);
+                    }
+                }
+            }
+
+            Ok(results.to_vector())
+        })
+        .join()
+        .map_err(|e| {
+            error!(e; "Join thread error");
+            ThreadJoin {
+                location: Location::default(),
+            }
+        })?
     }
 }
 
-fn signature() -> Signature {
-    Signature::one_of(
-        vec![
-            // migrate_region(region_id, from_peer, to_peer)
-            TypeSignature::Uniform(3, ConcreteDataType::numerics()),
-            // migrate_region(region_id, from_peer, to_peer, timeout(secs))
-            TypeSignature::Uniform(4, ConcreteDataType::numerics()),
-        ],
-        Volatility::Immutable,
-    )
+impl fmt::Display for MigrateRegionFunction {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        write!(f, "MIGRATE_REGION")
+    }
 }
 
 #[cfg(test)]
 mod tests {
-    use std::sync::Arc;
-
-    use common_query::prelude::TypeSignature;
-    use datatypes::vectors::{StringVector, UInt64Vector};
-
-    use super::*;
-
-    #[test]
-    fn test_migrate_region_misc() {
-        let f = MigrateRegionFunction;
-        assert_eq!("migrate_region", f.name());
-        assert_eq!(
-            ConcreteDataType::string_datatype(),
-            f.return_type(&[]).unwrap()
-        );
-        assert!(matches!(f.signature(),
-                         Signature {
-                             type_signature: TypeSignature::OneOf(sigs),
-                             volatility: Volatility::Immutable
-                         } if sigs.len() == 2));
-    }
-
-    #[test]
-    fn test_missing_procedure_service() {
-        let f = MigrateRegionFunction;
-
-        let args = vec![1, 1, 1];
-
-        let args = args
-            .into_iter()
-            .map(|arg| Arc::new(UInt64Vector::from_slice([arg])) as _)
-            .collect::<Vec<_>>();
-
-        let result = f.eval(FunctionContext::default(), &args).unwrap_err();
-        assert_eq!(
-            "Missing ProcedureServiceHandler, not expected",
-            result.to_string()
-        );
-    }
-
-    #[test]
-    fn test_migrate_region() {
-        let f = MigrateRegionFunction;
-
-        let args = vec![1, 1, 1];
-
-        let args = args
-            .into_iter()
-            .map(|arg| Arc::new(UInt64Vector::from_slice([arg])) as _)
-            .collect::<Vec<_>>();
-
-        let result = f.eval(FunctionContext::mock(), &args).unwrap();
-
-        let expect: VectorRef = Arc::new(StringVector::from(vec!["test_pid"]));
-        assert_eq!(expect, result);
-    }
+    // FIXME(dennis): test in the following PR.
 }

src/common/greptimedb-telemetry/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 async-trait.workspace = true
 common-error.workspace = true

src/common/grpc-expr/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 async-trait.workspace = true

src/common/grpc/Cargo.toml

@@ -4,9 +4,6 @@ version.workspace = true
 edition.workspace = true
 license.workspace = true
 
-[lints]
-workspace = true
-
 [dependencies]
 api.workspace = true
 arrow-flight.workspace = true

src/common/grpc/src/flight.rs

@@ -50,12 +50,8 @@ pub struct FlightEncoder {
 
 impl Default for FlightEncoder {
     fn default() -> Self {
-        let write_options = writer::IpcWriteOptions::default()
-            .try_with_compression(Some(arrow::ipc::CompressionType::LZ4_FRAME))
-            .unwrap();
-
         Self {
-            write_options,
+            write_options: writer::IpcWriteOptions::default(),
             data_gen: writer::IpcDataGenerator::default(),
             dictionary_tracker: writer::DictionaryTracker::new(false),
         }

src/common/macro/Cargo.toml

@@ -7,9 +7,6 @@ license.workspace = true
 [lib]
 proc-macro = true
 
-[lints]
-workspace = true
-
 [dependencies]
 proc-macro2 = "1.0.66"
 quote = "1.0"

src/common/macro/src/admin_fn.rs

@@ -1,236 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use proc_macro::TokenStream;
-use quote::quote;
-use syn::spanned::Spanned;
-use syn::{
-    parse_macro_input, Attribute, AttributeArgs, Ident, ItemFn, Signature, Type, TypePath,
-    TypeReference, Visibility,
-};
-
-use crate::utils::{extract_arg_map, extract_input_types, get_ident};
-
-/// Internal util macro to early return on error.
-macro_rules! ok {
-    ($item:expr) => {
-        match $item {
-            Ok(item) => item,
-            Err(e) => return e.into_compile_error().into(),
-        }
-    };
-}
-
-/// Internal util macro to create an error.
-macro_rules! error {
-    ($span:expr, $msg: expr) => {
-        Err(syn::Error::new($span, $msg))
-    };
-}
-
-pub(crate) fn process_admin_fn(args: TokenStream, input: TokenStream) -> TokenStream {
-    let mut result = TokenStream::new();
-
-    // extract arg map
-    let arg_pairs = parse_macro_input!(args as AttributeArgs);
-    let arg_span = arg_pairs[0].span();
-    let arg_map = ok!(extract_arg_map(arg_pairs));
-
-    // decompose the fn block
-    let compute_fn = parse_macro_input!(input as ItemFn);
-    let ItemFn {
-        attrs,
-        vis,
-        sig,
-        block,
-    } = compute_fn;
-
-    // extract fn arg list
-    let Signature {
-        inputs,
-        ident: fn_name,
-        ..
-    } = &sig;
-
-    let arg_types = ok!(extract_input_types(inputs));
-    if arg_types.len() < 2 {
-        ok!(error!(
-            sig.span(),
-            "Expect at least two argument for admin fn: (handler, query_ctx)"
-        ));
-    }
-    let handler_type = ok!(extract_handler_type(&arg_types));
-
-    // build the struct and its impl block
-    // only do this when `display_name` is specified
-    if let Ok(display_name) = get_ident(&arg_map, "display_name", arg_span) {
-        let struct_code = build_struct(
-            attrs,
-            vis,
-            fn_name,
-            ok!(get_ident(&arg_map, "name", arg_span)),
-            ok!(get_ident(&arg_map, "sig_fn", arg_span)),
-            ok!(get_ident(&arg_map, "ret", arg_span)),
-            handler_type,
-            display_name,
-        );
-        result.extend(struct_code);
-    }
-
-    // preserve this fn
-    let input_fn_code: TokenStream = quote! {
-        #sig { #block }
-    }
-    .into();
-
-    result.extend(input_fn_code);
-    result
-}
-
-/// Retrieve the handler type, `ProcedureServiceHandlerRef` or `TableMutationHandlerRef`.
-fn extract_handler_type(arg_types: &[Type]) -> Result<&Ident, syn::Error> {
-    match &arg_types[0] {
-        Type::Reference(TypeReference { elem, .. }) => match &**elem {
-            Type::Path(TypePath { path, .. }) => Ok(&path
-                .segments
-                .first()
-                .expect("Expected a reference of handler")
-                .ident),
-            other => {
-                error!(other.span(), "Expected a reference of handler")
-            }
-        },
-        other => {
-            error!(other.span(), "Expected a reference of handler")
-        }
-    }
-}
-
-/// Build the function struct
-#[allow(clippy::too_many_arguments)]
-fn build_struct(
-    attrs: Vec<Attribute>,
-    vis: Visibility,
-    fn_name: &Ident,
-    name: Ident,
-    sig_fn: Ident,
-    ret: Ident,
-    handler_type: &Ident,
-    display_name_ident: Ident,
-) -> TokenStream {
-    let display_name = display_name_ident.to_string();
-    let ret = Ident::new(&format!("{ret}_datatype"), ret.span());
-    let uppcase_display_name = display_name.to_uppercase();
-    // Get the handler name in function state by the argument ident
-    let (handler, snafu_type) = match handler_type.to_string().as_str() {
-        "ProcedureServiceHandlerRef" => (
-            Ident::new("procedure_service_handler", handler_type.span()),
-            Ident::new("MissingProcedureServiceHandlerSnafu", handler_type.span()),
-        ),
-
-        "TableMutationHandlerRef" => (
-            Ident::new("table_mutation_handler", handler_type.span()),
-            Ident::new("MissingTableMutationHandlerSnafu", handler_type.span()),
-        ),
-        handler => ok!(error!(
-            handler_type.span(),
-            format!("Unknown handler type: {handler}")
-        )),
-    };
-
-    quote! {
-        #(#attrs)*
-        #[derive(Debug)]
-        #vis struct #name;
-
-        impl fmt::Display for #name {
-            fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-                write!(f, #uppcase_display_name)
-            }
-        }
-
-
-        impl Function for #name {
-            fn name(&self) -> &'static str {
-                #display_name
-            }
-
-            fn return_type(&self, _input_types: &[ConcreteDataType]) -> Result<ConcreteDataType> {
-                Ok(ConcreteDataType::#ret())
-            }
-
-            fn signature(&self) -> Signature {
-                #sig_fn()
-            }
-
-            fn eval(&self, func_ctx: FunctionContext, columns: &[VectorRef]) ->  Result<VectorRef> {
-                // Ensure under the `greptime` catalog for security
-                ensure_greptime!(func_ctx);
-
-                let columns_num = columns.len();
-                let rows_num = if columns.is_empty() {
-                    1
-                } else {
-                    columns[0].len()
-                };
-                let columns = Vec::from(columns);
-
-                // TODO(dennis): DataFusion doesn't support async UDF currently
-                std::thread::spawn(move || {
-                    let query_ctx = &func_ctx.query_ctx;
-                    let handler = func_ctx
-                        .state
-                        .#handler
-                        .as_ref()
-                        .context(#snafu_type)?;
-
-                    let mut builder = ConcreteDataType::#ret()
-                        .create_mutable_vector(rows_num);
-
-                    if columns_num == 0 {
-                        let result = common_runtime::block_on_read(async move {
-                            #fn_name(handler, query_ctx, &[]).await
-                        })?;
-
-                        builder.push_value_ref(result.as_value_ref());
-                    } else {
-                        for i in 0..rows_num {
-                            let args: Vec<_> = columns.iter()
-                                .map(|vector| vector.get_ref(i))
-                                .collect();
-
-                            let result = common_runtime::block_on_read(async move {
-                                #fn_name(handler, query_ctx, &args).await
-                            })?;
-
-                            builder.push_value_ref(result.as_value_ref());
-                        }
-                    }
-
-                    Ok(builder.to_vector())
-                })
-                    .join()
-                    .map_err(|e| {
-                        error!(e; "Join thread error");
-                        ThreadJoin {
-                            location: Location::default(),
-                        }
-                    })?
-
-            }
-
-        }
-    }
-    .into()
-}

src/common/macro/src/lib.rs

@@ -12,20 +12,17 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-mod admin_fn;
 mod aggr_func;
 mod print_caller;
 mod range_fn;
 mod stack_trace_debug;
-mod utils;
+
 use aggr_func::{impl_aggr_func_type_store, impl_as_aggr_func_creator};
 use print_caller::process_print_caller;
 use proc_macro::TokenStream;
 use range_fn::process_range_fn;
 use syn::{parse_macro_input, DeriveInput};
 
-use crate::admin_fn::process_admin_fn;
-
 /// Make struct implemented trait [AggrFuncTypeStore], which is necessary when writing UDAF.
 /// This derive macro is expect to be used along with attribute macro [macro@as_aggr_func_creator].
 #[proc_macro_derive(AggrFuncTypeStore)]
@@ -71,25 +68,6 @@ pub fn range_fn(args: TokenStream, input: TokenStream) -> TokenStream {
     process_range_fn(args, input)
 }
 
-/// Attribute macro to convert a normal function to SQL administration function. The annotated function
-/// should accept:
-///    - `&ProcedureServiceHandlerRef` or `&TableMutationHandlerRef` as the first argument,
-///    - `&QueryContextRef` as the second argument, and
-///    - `&[ValueRef<'_>]` as the third argument which is SQL function input values in each row.
-/// Return type must be `common_query::error::Result<Value>`.
-///
-/// # Example see `common/function/src/system/procedure_state.rs`.
-///
-/// # Arguments
-/// - `name`: The name of the generated `Function` implementation.
-/// - `ret`: The return type of the generated SQL function, it will be transformed into `ConcreteDataType::{ret}_datatype()` result.
-/// - `display_name`: The display name of the generated SQL function.
-/// - `sig_fn`: the function to returns `Signature` of generated `Function`.
-#[proc_macro_attribute]
-pub fn admin_fn(args: TokenStream, input: TokenStream) -> TokenStream {
-    process_admin_fn(args, input)
-}
-
 /// Attribute macro to print the caller to the annotated function.
 /// The caller is printed as its filename and the call site line number.
 ///

src/common/macro/src/range_fn.rs

@@ -12,16 +12,20 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+use std::collections::HashMap;
+
 use proc_macro::TokenStream;
+use proc_macro2::Span;
 use quote::quote;
+use syn::punctuated::Punctuated;
 use syn::spanned::Spanned;
+use syn::token::Comma;
 use syn::{
-    parse_macro_input, Attribute, AttributeArgs, Ident, ItemFn, Signature, Type, TypeReference,
-    Visibility,
+    parse_macro_input, Attribute, AttributeArgs, FnArg, Ident, ItemFn, Meta, MetaNameValue,
+    NestedMeta, Signature, Type, TypeReference, Visibility,
 };
 
-use crate::utils::{extract_arg_map, extract_input_types, get_ident};
-
+/// Internal util macro to early return on error.
 macro_rules! ok {
     ($item:expr) => {
         match $item {
@@ -56,18 +60,6 @@ pub(crate) fn process_range_fn(args: TokenStream, input: TokenStream) -> TokenSt
     } = &sig;
     let arg_types = ok!(extract_input_types(inputs));
 
-    // with format like Float64Array
-    let array_types = arg_types
-        .iter()
-        .map(|ty| {
-            if let Type::Reference(TypeReference { elem, .. }) = ty {
-                elem.as_ref().clone()
-            } else {
-                ty.clone()
-            }
-        })
-        .collect::<Vec<_>>();
-
     // build the struct and its impl block
     // only do this when `display_name` is specified
     if let Ok(display_name) = get_ident(&arg_map, "display_name", arg_span) {
@@ -76,8 +68,6 @@ pub(crate) fn process_range_fn(args: TokenStream, input: TokenStream) -> TokenSt
             vis,
             ok!(get_ident(&arg_map, "name", arg_span)),
             display_name,
-            array_types,
-            ok!(get_ident(&arg_map, "ret", arg_span)),
         );
         result.extend(struct_code);
     }
@@ -99,13 +89,53 @@ pub(crate) fn process_range_fn(args: TokenStream, input: TokenStream) -> TokenSt
     result
 }
 
+/// Extract a String <-> Ident map from the attribute args.
+fn extract_arg_map(args: Vec<NestedMeta>) -> Result<HashMap<String, Ident>, syn::Error> {
+    args.into_iter()
+        .map(|meta| {
+            if let NestedMeta::Meta(Meta::NameValue(MetaNameValue { path, lit, .. })) = meta {
+                let name = path.get_ident().unwrap().to_string();
+                let ident = match lit {
+                    syn::Lit::Str(lit_str) => lit_str.parse::<Ident>(),
+                    _ => Err(syn::Error::new(
+                        lit.span(),
+                        "Unexpected attribute format. Expected `name = \"value\"`",
+                    )),
+                }?;
+                Ok((name, ident))
+            } else {
+                Err(syn::Error::new(
+                    meta.span(),
+                    "Unexpected attribute format. Expected `name = \"value\"`",
+                ))
+            }
+        })
+        .collect::<Result<HashMap<String, Ident>, syn::Error>>()
+}
+
+/// Helper function to get an Ident from the previous arg map.
+fn get_ident(map: &HashMap<String, Ident>, key: &str, span: Span) -> Result<Ident, syn::Error> {
+    map.get(key)
+        .cloned()
+        .ok_or_else(|| syn::Error::new(span, format!("Expect attribute {key} but not found")))
+}
+
+/// Extract the argument list from the annotated function.
+fn extract_input_types(inputs: &Punctuated<FnArg, Comma>) -> Result<Vec<Type>, syn::Error> {
+    inputs
+        .iter()
+        .map(|arg| match arg {
+            FnArg::Receiver(receiver) => Err(syn::Error::new(receiver.span(), "expected bool")),
+            FnArg::Typed(pat_type) => Ok(*pat_type.ty.clone()),
+        })
+        .collect()
+}
+
 fn build_struct(
     attrs: Vec<Attribute>,
     vis: Visibility,
     name: Ident,
     display_name_ident: Ident,
-    array_types: Vec<Type>,
-    return_array_type: Ident,
 ) -> TokenStream {
     let display_name = display_name_ident.to_string();
     quote! {
@@ -130,12 +160,18 @@ fn build_struct(
                 }
             }
 
+            // TODO(ruihang): this should be parameterized
+            // time index column and value column
             fn input_type() -> Vec<DataType> {
-                vec![#( RangeArray::convert_data_type(#array_types::new_null(0).data_type().clone()), )*]
+                vec![
+                    RangeArray::convert_data_type(DataType::Timestamp(TimeUnit::Millisecond, None)),
+                    RangeArray::convert_data_type(DataType::Float64),
+                ]
             }
 
+            // TODO(ruihang): this should be parameterized
             fn return_type() -> DataType {
-                #return_array_type::new_null(0).data_type().clone()
+                DataType::Float64
             }
         }
     }
@@ -170,7 +206,6 @@ fn build_calc_fn(
         .map(|name| Ident::new(&format!("{}_range_array", name), name.span()))
         .collect::<Vec<_>>();
     let first_range_array_name = range_array_names.first().unwrap().clone();
-    let first_param_name = param_names.first().unwrap().clone();
 
     quote! {
         impl #name {
@@ -179,29 +214,13 @@ fn build_calc_fn(
 
                 #( let #range_array_names = RangeArray::try_new(extract_array(&input[#param_numbers])?.to_data().into())?; )*
 
-                // check arrays len
-                {
-                    let len_first = #first_range_array_name.len();
-                    #(
-                        if len_first != #range_array_names.len() {
-                            return Err(DataFusionError::Execution(format!("RangeArray have different lengths in PromQL function {}: array1={}, array2={}", #name::name(), len_first, #range_array_names.len())));
-                        }
-                    )*
-                }
+                // TODO(ruihang): add ensure!() 
 
                 let mut result_array = Vec::new();
                 for index in 0..#first_range_array_name.len(){
                     #( let #param_names = #range_array_names.get(index).unwrap().as_any().downcast_ref::<#unref_param_types>().unwrap().clone(); )*
 
-                    // check element len
-                    {
-                        let len_first = #first_param_name.len();
-                        #(
-                            if len_first != #param_names.len() {
-                                return Err(DataFusionError::Execution(format!("RangeArray's element {} have different lengths in PromQL function {}: array1={}, array2={}", index, #name::name(), len_first, #param_names.len())));
-                            }
-                        )*
-                    }
+                    // TODO(ruihang): add ensure!() to check length
 
                     let result = #fn_name(#( &#param_names, )*);
                     result_array.push(result);

src/common/macro/src/utils.rs

@@ -1,69 +0,0 @@
-// Copyright 2023 Greptime Team
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-use std::collections::HashMap;
-
-use proc_macro2::Span;
-use syn::punctuated::Punctuated;
-use syn::spanned::Spanned;
-use syn::token::Comma;
-use syn::{FnArg, Ident, Meta, MetaNameValue, NestedMeta, Type};
-
-/// Extract a String <-> Ident map from the attribute args.
-pub(crate) fn extract_arg_map(args: Vec<NestedMeta>) -> Result<HashMap<String, Ident>, syn::Error> {
-    args.into_iter()
-        .map(|meta| {
-            if let NestedMeta::Meta(Meta::NameValue(MetaNameValue { path, lit, .. })) = meta {
-                let name = path.get_ident().unwrap().to_string();
-                let ident = match lit {
-                    syn::Lit::Str(lit_str) => lit_str.parse::<Ident>(),
-                    _ => Err(syn::Error::new(
-                        lit.span(),
-                        "Unexpected attribute format. Expected `name = \"value\"`",
-                    )),
-                }?;
-                Ok((name, ident))
-            } else {
-                Err(syn::Error::new(
-                    meta.span(),
-                    "Unexpected attribute format. Expected `name = \"value\"`",
-                ))
-            }
-        })
-        .collect::<Result<HashMap<String, Ident>, syn::Error>>()
-}
-
-/// Helper function to get an Ident from the previous arg map.
-pub(crate) fn get_ident(
-    map: &HashMap<String, Ident>,
-    key: &str,
-    span: Span,
-) -> Result<Ident, syn::Error> {
-    map.get(key)
-        .cloned()
-        .ok_or_else(|| syn::Error::new(span, format!("Expect attribute {key} but not found")))
-}
-
-/// Extract the argument list from the annotated function.
-pub(crate) fn extract_input_types(
-    inputs: &Punctuated<FnArg, Comma>,
-) -> Result<Vec<Type>, syn::Error> {
-    inputs
-        .iter()
-        .map(|arg| match arg {
-            FnArg::Receiver(receiver) => Err(syn::Error::new(receiver.span(), "expected bool")),
-            FnArg::Typed(pat_type) => Ok(*pat_type.ty.clone()),
-        })
-        .collect()
-}