Merge pull request #10628 from everpcpc/fix-log

chore(query): do not write duplicate query log

Author: BohuTANG

.github/actions/artifact_download/action.yml

@@ -46,7 +46,7 @@ runs:
         fi
         echo "src=${src}" >> $GITHUB_OUTPUT
 
-    - uses: actions/download-artifact@v2
+    - uses: actions/download-artifact@v3
       if: steps.info.outputs.src == 'github'
       with:
         name: ${{ inputs.profile }}-${{ inputs.sha }}-${{ inputs.target }}-${{ inputs.category }}

.github/workflows/backport.yml

@@ -77,7 +77,7 @@ jobs:
           dockerhub_user: ${{ secrets.DOCKERHUB_USERNAME }}
           dockerhub_token: ${{ secrets.DOCKERHUB_TOKEN }}
       - name: Download binaries
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: databend
           path: distro

.github/workflows/release.yml

@@ -496,7 +496,7 @@ jobs:
           prerelease: true
           latest: true
       - name: download sha256sums
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v3
         with:
           name: sha256sums
       - shell: bash

src/query/service/src/interpreters/interpreter_query_log.rs

@@ -60,15 +60,14 @@ fn error_fields(log_type: LogType, err: Option<ErrorCode>) -> (LogType, i32, Str
 
 impl InterpreterQueryLog {
     fn write_log(event: QueryLogElement) -> Result<()> {
-        info!("{}", serde_json::to_string(&event)?);
-
+        let event_str = serde_json::to_string(&event)?;
         if let Some(logger) = QueryLogger::instance().get_subscriber() {
-            let event_str = serde_json::to_string(&event)?;
             subscriber::with_default(logger, || {
                 info!("{}", event_str);
             });
+        } else {
+            info!("{}", event_str);
         };
-
         QueryLogQueue::instance()?.append_data(event)
     }