Merge branch 'main' into issue-7565

Author: mergify[bot]

.github/actions/test_sqllogic_cluster_linux/action.yml

@@ -23,12 +23,13 @@ runs:
         sha: ${{ github.sha }}
         target: ${{ inputs.target }}
 
-    - name: Test setup
-      shell: bash
-      run: |
-        bash ./scripts/setup/dev_setup.sh -yd
-
     - name: Run sqllogic Tests with Cluster mode
       shell: bash
       run: |
-        bash ./scripts/ci/ci-run-sqllogic-tests-cluster.sh ${{ inputs.dirs }}
+        docker run --rm --tty --net=host \
+          --user $(id -u):$(id -g) \
+          --volume "${PWD}:/workspace" \
+          --workdir "/workspace" \
+          datafuselabs/build-tool:sqllogic \
+          bash ./scripts/ci/ci-run-sqllogic-tests-cluster.sh \
+          ${{ inputs.dirs }}

.github/actions/test_sqllogic_standalone_linux/action.yml

@@ -23,18 +23,13 @@ runs:
         sha: ${{ github.sha }}
         target: ${{ inputs.target }}
 
-    - name: Test setup
-      shell: bash
-      run: |
-        bash ./scripts/setup/dev_setup.sh -yd
-
     - name: Run sqllogic Tests with Standalone mode with embedded meta-store
       shell: bash
       run: |
-        bash ./scripts/ci/ci-run-sqllogic-tests.sh ${{ inputs.dirs }}
-
-    # - name: Upload failure
-    #   if: failure()
-    #   uses: ./.github/actions/artifact_failure
-    #   with:
-    #     name: test-sqllogic-standalone-linux
+        docker run --rm --tty --net=host \
+          --user $(id -u):$(id -g) \
+          --volume "${PWD}:/workspace" \
+          --workdir "/workspace" \
+          datafuselabs/build-tool:sqllogic \
+          bash ./scripts/ci/ci-run-sqllogic-tests.sh \
+          ${{ inputs.dirs }}

.github/workflows/databend-release.yml

@@ -203,7 +203,7 @@ jobs:
         run: echo ::set-output name=VERSION::${GITHUB_REF/refs\/tags\//}
       - name: Upload to github release
         env:
-          GH_TOKEN: ${{ inputs.github_token }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
           version=${{ needs.create_release.outputs.version }}
           tar -C ./tests/logictest -czvf testsuites-${version}.tar.gz suites

src/query/legacy-parser/src/parser/expr_parser.rs

@@ -22,14 +22,6 @@ use sqlparser::tokenizer::Tokenizer;
 pub struct ExprParser;
 
 impl ExprParser {
-    pub fn parse_expr(expr: &str) -> Result<Expr, ParserError> {
-        let dialect = &MySqlDialect {};
-        let mut tokenizer = Tokenizer::new(dialect, expr);
-        let (tokens, position_map) = tokenizer.tokenize()?;
-        let mut parser = Parser::new(tokens, position_map, dialect);
-        parser.parse_expr()
-    }
-
     pub fn parse_exprs(expr: &str) -> Result<Vec<Expr>, ParserError> {
         let dialect = &MySqlDialect {};
         let mut tokenizer = Tokenizer::new(dialect, expr);

src/query/legacy-parser/src/parser/expression_parser.rs

@@ -21,12 +21,6 @@ use crate::ExprParser;
 pub struct ExpressionParser;
 
 impl ExpressionParser {
-    pub fn parse_expr(expr: &str) -> Result<Expression> {
-        let expr = ExprParser::parse_expr(expr)?;
-        let analyzer = ExpressionSyncAnalyzer::create();
-        analyzer.analyze(&expr)
-    }
-
     pub fn parse_exprs(expr: &str) -> Result<Vec<Expression>> {
         let exprs = ExprParser::parse_exprs(expr)?;
         let analyzer = ExpressionSyncAnalyzer::create();

src/query/planners/src/lib.rs

@@ -82,7 +82,6 @@ mod plan_sink;
 mod plan_sort;
 mod plan_subqueries_set;
 mod plan_table_alter_cluster_key;
-mod plan_table_create;
 mod plan_table_describe;
 mod plan_table_drop;
 mod plan_table_drop_cluster_key;
@@ -222,8 +221,6 @@ pub use plan_sink::SINK_SCHEMA;
 pub use plan_sort::SortPlan;
 pub use plan_subqueries_set::SubQueriesSetPlan;
 pub use plan_table_alter_cluster_key::AlterTableClusterKeyPlan;
-pub use plan_table_create::CreateTablePlan;
-pub use plan_table_create::TableOptions;
 pub use plan_table_describe::DescribeTablePlan;
 pub use plan_table_drop::DropTablePlan;
 pub use plan_table_drop_cluster_key::DropTableClusterKeyPlan;