fix: windows build and check ci check for windows (#2819)

GreptimeTeam · Nov 27, 2023 · bd9c2f2 · bd9c2f2
1 parent b3edbef
commit bd9c2f2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.github/workflows/develop.yml b/.github/workflows/develop.yml
@@ -42,7 +42,10 @@ jobs:
   check:
     name: Check
     if: github.event.pull_request.draft == false
-    runs-on: ubuntu-20.04
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ windows-latest-8-cores, ubuntu-20.04 ]
     timeout-minutes: 60
     steps:
       - uses: actions/checkout@v3

diff --git a/src/servers/src/error.rs b/src/servers/src/error.rs
@@ -339,6 +339,7 @@ pub enum Error {
         source: crate::http::pprof::nix::Error,
     },
 
+    #[cfg(not(windows))]
     #[snafu(display("Failed to update jemalloc metrics"))]
     UpdateJemallocMetrics {
         #[snafu(source)]
@@ -412,8 +413,10 @@ impl ErrorExt for Error {
             | TcpIncoming { .. }
             | CatalogError { .. }
             | GrpcReflectionService { .. }
-            | BuildHttpResponse { .. }
-            | UpdateJemallocMetrics { .. } => StatusCode::Internal,
+            | BuildHttpResponse { .. } => StatusCode::Internal,
+
+            #[cfg(not(windows))]
+            UpdateJemallocMetrics { .. } => StatusCode::Internal,
 
             CollectRecordbatch { .. } => StatusCode::EngineExecuteQuery,